[Activity log] Generalize the whitelist for keeping arguments

chrome/browser/extensions/activity_log/activity_log_policy.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_ACTIVITY_LOG_POLICY_H_
 #define CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_ACTIVITY_LOG_POLICY_H_
 
+#include <map>
 #include <set>
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/activity_log/activity_actions.h"
 #include "chrome/browser/extensions/activity_log/activity_database.h"
@@ skipping 79 matching lines @@
 
   // For unit testing only.
   void SetClockForTesting(scoped_ptr<base::Clock> clock);
 
   // A collection of methods that are useful for implementing policies.  These
   // are all static methods; the ActivityLogPolicy::Util class cannot be
   // instantiated.  This is nested within ActivityLogPolicy to make calling
   // these methods more convenient from within a policy, but they are public.
   class Util {
    public:
+    // A collection of API calls, used to specify whitelists for argument
+    // filtering.
+    typedef std::set<std::pair<Action::ActionType, std::string> > ApiSet;
+
     // Serialize a Value as a JSON string.  Returns an empty string if value is
     // null.
     static std::string Serialize(const base::Value* value);
 
     // Removes potentially privacy-sensitive data that should not be logged.
     // This should generally be called on an Action before logging, unless
     // debugging flags are enabled.  Modifies the Action object in place; if
     // the action might be shared with other users, it is up to the caller to
     // call ->Clone() first.
     static void StripPrivacySensitiveFields(scoped_refptr<Action> action);
 
     // Strip arguments from most API actions, preserving actions only for a
     // whitelisted set.  Modifies the Action object in-place.
-    static void StripArguments(const std::set<std::string>& api_whitelist,
+    static void StripArguments(const ApiSet& api_whitelist,
                                scoped_refptr<Action> action);
 
     // Given a base day (timestamp at local midnight), computes the timestamp
     // at midnight the given number of days before or after.
     static base::Time AddDays(const base::Time& base_date, int days);
 
     // Compute the time bounds that should be used for a database query to
     // cover a time range days_ago days in the past, relative to the specified
     // time.
     static void ComputeDatabaseTimeBounds(const base::Time& now,
@@ skipping 81 matching lines @@
 
  private:
   // See the comments for the ActivityDatabase class for a discussion of how
   // database cleanup runs.
   ActivityDatabase* db_;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_ACTIVITY_LOG_POLICY_H_