[UMA] Use proper C++ objects to serialize tracked_objects across process boundaries.

content/browser/profiler_controller_impl.h

Index: content/browser/profiler_controller_impl.h
diff --git a/content/browser/profiler_controller_impl.h b/content/browser/profiler_controller_impl.h
index ff361ea6e504834a4e65ec78659a0714cd9983a5..e3e8d8549d5f93fbeb212ee8478a3701af848904 100644
--- a/content/browser/profiler_controller_impl.h
+++ b/content/browser/profiler_controller_impl.h
@@ -6,14 +6,15 @@
 #define CONTENT_BROWSER_PROFILER_CONTROLLER_IMPL_H_
 
 #include "base/memory/singleton.h"
-#include "base/tracked_objects.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/profiler_controller.h"
 
 namespace content {
 
+struct SerializedProfilerData;
+
 // ProfilerController's implementation.
-class CONTENT_EXPORT ProfilerControllerImpl : public ProfilerController {
+class ProfilerControllerImpl : public ProfilerController {
  public:
   static ProfilerControllerImpl* GetInstance();
 
@@ -22,21 +23,19 @@ class CONTENT_EXPORT ProfilerControllerImpl : public ProfilerController {
   ProfilerControllerImpl();
   virtual ~ProfilerControllerImpl();
 
-  // Send number of pending processes to subscriber_. |end| is set to true if it
-  // is the last time. This is called on UI thread.
+  // Send the number of |pending_processes| to |subscriber_|. |end| is set to
+  // true if it is the last time. This is called on the UI thread.
   void OnPendingProcesses(int sequence_number, int pending_processes, bool end);
 
-  // Send profiler_data back to subscriber_. subscriber_ assumes the ownership
-  // of profiler_data. This is called on UI thread.
+  // Send the |profiler_data| back to the |subscriber_|.
+  // This is called on the UI thread.
   void OnProfilerDataCollected(int sequence_number,
-                               base::DictionaryValue* profiler_data);
+                               const SerializedProfilerData& profiler_data);
 
   // ProfilerController implementation:
   virtual void Register(ProfilerSubscriber* subscriber) OVERRIDE;
-  virtual void Unregister(ProfilerSubscriber* subscriber) OVERRIDE;
+  virtual void Unregister(const ProfilerSubscriber* subscriber) OVERRIDE;
   virtual void GetProfilerData(int sequence_number) OVERRIDE;
-  virtual void SetProfilerStatus(
-      tracked_objects::ThreadData::Status status) OVERRIDE;
 
  private:
   friend struct DefaultSingletonTraits<ProfilerControllerImpl>;
@@ -44,10 +43,6 @@ class CONTENT_EXPORT ProfilerControllerImpl : public ProfilerController {
   // Contact child processes and get their profiler data.
   void GetProfilerDataFromChildProcesses(int sequence_number);
 
-  // Contact child processes and set profiler status to |enable|.
-  void SetProfilerStatusInChildProcesses(
-      tracked_objects::ThreadData::Status status);
-
   ProfilerSubscriber* subscriber_;
 
   DISALLOW_COPY_AND_ASSIGN(ProfilerControllerImpl);
@@ -56,4 +51,3 @@ class CONTENT_EXPORT ProfilerControllerImpl : public ProfilerController {
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_PROFILER_CONTROLLER_IMPL_H_
-