[Activity log] Generalize the whitelist for keeping arguments

chrome/browser/extensions/activity_log/activity_log_policy_unittest.cc

Index: chrome/browser/extensions/activity_log/activity_log_policy_unittest.cc
diff --git a/chrome/browser/extensions/activity_log/activity_log_policy_unittest.cc b/chrome/browser/extensions/activity_log/activity_log_policy_unittest.cc
index 9ad74ae0ea7f02f84f310c089ecad28094411fa9..0d64928cfee78bb69adca822399bc156087b5fc5 100644
--- a/chrome/browser/extensions/activity_log/activity_log_policy_unittest.cc
+++ b/chrome/browser/extensions/activity_log/activity_log_policy_unittest.cc
@@ -59,8 +59,9 @@ TEST_F(ActivityLogPolicyUtilTest, StripPrivacySensitiveWebRequest) {
 
 // Test that argument values are stripped as appropriate.
 TEST_F(ActivityLogPolicyUtilTest, StripArguments) {
-  std::set<std::string> whitelist;
-  whitelist.insert("tabs.executeScript");
+  ActivityLogPolicy::Util::ApiSet whitelist;
+  whitelist.insert(
+      std::make_pair(Action::ACTION_API_CALL, "tabs.executeScript"));
 
   // API is in whitelist; not stripped.
   scoped_refptr<Action> action =
@@ -78,13 +79,6 @@ TEST_F(ActivityLogPolicyUtilTest, StripArguments) {
   action->mutable_args()->AppendString("woof");
   ActivityLogPolicy::Util::StripArguments(whitelist, action);
   ASSERT_EQ("", ActivityLogPolicy::Util::Serialize(action->args()));
-
-  // Not an API type: not stripped.
-  action = new Action(
-      "punky", base::Time::Now(), Action::ACTION_DOM_ACCESS, "tabs.create");
-  action->mutable_args()->AppendString("woof");
-  ActivityLogPolicy::Util::StripArguments(whitelist, action);
-  ASSERT_EQ("[\"woof\"]", ActivityLogPolicy::Util::Serialize(action->args()));
 }
 
 // Test parsing of URLs serialized to strings.