Hooked the trace event argument whitelist up to the background_trace_manager

content/public/browser/background_tracing_manager.h

Index: content/public/browser/background_tracing_manager.h
diff --git a/content/public/browser/background_tracing_manager.h b/content/public/browser/background_tracing_manager.h
index d5e0900f161194badf775087ec0d3bc225de9f24..81823c4dc3020eca6f691def6eef3a71db7c8708 100644
--- a/content/public/browser/background_tracing_manager.h
+++ b/content/public/browser/background_tracing_manager.h
@@ -39,8 +39,8 @@ class BackgroundTracingManager {
   //     );
   // }
   //
-  typedef base::Callback<void(const base::RefCountedString*, base::Closure)>
-      ReceiveCallback;
+  typedef base::Callback<void(const scoped_refptr<base::RefCountedString>&,
+                              base::Closure)> ReceiveCallback;
 
   // Set the triggering rules for when to start recording.
   //
@@ -57,9 +57,13 @@ class BackgroundTracingManager {
   // Calls to SetActiveScenario() with a config will fail if tracing is
   // currently on. Use WhenIdle to register a callback to get notified when
   // the manager is idle and a config can be set again.
+  enum DataFiltering {
+    NO_DATA_FILTERING,
+    ANONYMIZE_DATA,
+  };
   virtual bool SetActiveScenario(scoped_ptr<BackgroundTracingConfig> config,
                                  const ReceiveCallback& receive_callback,
-                                 bool requires_anonymized_data) = 0;
+                                 DataFiltering data_filtering) = 0;

[davidben, 2015/06/02 22:52:02]

For a follow-up since this was like this before: I believe methods of Foo in
//content that are only called elsewhere in //content just get put on the
FooImpl and then we cast to FooImpl.

 
   // Notifies the caller when the manager is idle (not recording or uploading),
   // so that a call to SetActiveScenario() is likely to succeed.
@@ -84,6 +88,8 @@ class BackgroundTracingManager {
   virtual void GetTriggerNameList(std::vector<std::string>* trigger_names) = 0;
 
   virtual void InvalidateTriggerHandlesForTesting() = 0;
+  virtual void SetTracingEnabledCallbackForTesting(
+      const base::Closure& callback) = 0;
 
  protected:
   virtual ~BackgroundTracingManager() {}