Hooked the trace event argument whitelist up to the background_trace_manager

content/browser/tracing/background_tracing_manager_impl.h

Index: content/browser/tracing/background_tracing_manager_impl.h
diff --git a/content/browser/tracing/background_tracing_manager_impl.h b/content/browser/tracing/background_tracing_manager_impl.h
index fd1fd00ad9e4fe147d0861e37c1327448ab6b53b..bb44217ba4fe4fc96dacfe85dad9e59441ea9ff6 100644
--- a/content/browser/tracing/background_tracing_manager_impl.h
+++ b/content/browser/tracing/background_tracing_manager_impl.h
@@ -21,6 +21,7 @@ class BackgroundTracingManagerImpl : public content::BackgroundTracingManager {
 
   bool SetActiveScenario(scoped_ptr<BackgroundTracingConfig>,
                          const ReceiveCallback&,
+                         const base::Closure& enabled_callback,
                          bool) override;

[davidben, 2015/06/05 16:40:06]

Style: parameters should be named. I don't think Chromium omits them like Blink
does.

https://google.github.io/styleguide/cppguide.html#Function_Declarations_and_D...

   void WhenIdle(IdleCallback idle_callback) override;
 
@@ -34,8 +35,9 @@ class BackgroundTracingManagerImpl : public content::BackgroundTracingManager {
   BackgroundTracingManagerImpl();
   ~BackgroundTracingManagerImpl() override;
 
-  void EnableRecording(base::trace_event::CategoryFilter);
-  void EnableRecordingIfConfigNeedsIt();
+  void EnableRecording(base::trace_event::CategoryFilter,
+                       const base::Closure& enabled_callback);
+  void EnableRecordingIfConfigNeedsIt(const base::Closure& enabled_callback);
   void OnFinalizeStarted(scoped_refptr<base::RefCountedString>);
   void OnFinalizeComplete();
   void BeginFinalizing(StartedFinalizingCallback);