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

chrome/browser/ui/webui/profiler_ui.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/ui/webui/profiler_ui.h"
 
 #include <string>
 
 // When testing the javacript code, it is cumbersome to have to keep
 // re-building the resouces package and reloading the browser. To solve
 // this, enable the following flag to read the webapp's source files
 // directly off disk, so all you have to do is refresh the page to
 // test the modifications.
 //#define USE_SOURCE_FILES_DIRECTLY
 
 #include "base/bind.h"
+#include "base/memory/scoped_ptr.h"
 #include "base/tracked_objects.h"
+#include "base/values.h"
 #include "chrome/browser/metrics/tracking_synchronizer.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/task_profiler/task_profiler_data_serializer.h"
 #include "chrome/browser/ui/webui/chrome_web_ui_data_source.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_ui.h"
 #include "content/public/browser/web_ui_message_handler.h"
 #include "grit/browser_resources.h"
 #include "grit/generated_resources.h"
 
 #ifdef USE_SOURCE_FILES_DIRECTLY
@@ skipping 90 matching lines @@
   void OnResetData(const ListValue* list);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ProfilerMessageHandler);
 };
 
 void ProfilerMessageHandler::RegisterMessages() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   web_ui()->RegisterMessageCallback("getData",
-      base::Bind(&ProfilerMessageHandler::OnGetData,base::Unretained(this)));
+      base::Bind(&ProfilerMessageHandler::OnGetData, base::Unretained(this)));
   web_ui()->RegisterMessageCallback("resetData",
       base::Bind(&ProfilerMessageHandler::OnResetData,
                  base::Unretained(this)));
 }
 
 void ProfilerMessageHandler::OnGetData(const ListValue* list) {
   ProfilerUI* profiler_ui = static_cast<ProfilerUI*>(web_ui()->GetController());
   profiler_ui->GetData();
 }
 
 void ProfilerMessageHandler::OnResetData(const ListValue* list) {
   tracked_objects::ThreadData::ResetAllThreadData();
 }
 
 }  // namespace
 
-ProfilerUI::ProfilerUI(content::WebUI* web_ui) : WebUIController(web_ui) {
-  ui_weak_ptr_factory_.reset(new base::WeakPtrFactory<ProfilerUI>(this));
-  ui_weak_ptr_ = ui_weak_ptr_factory_->GetWeakPtr();
-
+ProfilerUI::ProfilerUI(content::WebUI* web_ui)
+    : WebUIController(web_ui),
+      ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)) {
   web_ui->AddMessageHandler(new ProfilerMessageHandler());
 
   // Set up the chrome://profiler/ source.
   Profile::FromWebUI(web_ui)->
       GetChromeURLDataManager()->AddDataSource(CreateProfilerHTMLSource());
 }
 
 ProfilerUI::~ProfilerUI() {
 }
 
 void ProfilerUI::GetData() {
-  TrackingSynchronizer::FetchProfilerDataAsynchronously(ui_weak_ptr_);
+  TrackingSynchronizer::FetchProfilerDataAsynchronously(
+      weak_ptr_factory_.GetWeakPtr());
 }
 
-void ProfilerUI::ReceivedData(base::Value* value) {
+void ProfilerUI::ReceivedProfilerData(
+    const tracked_objects::ProcessDataSnapshot& profiler_data,
+    content::ProcessType process_type) {
+  // Serialize the data to JSON.
+  DictionaryValue json_data;
+  task_profiler::TaskProfilerDataSerializer::ToValue(profiler_data,
+                                                     process_type,
+                                                     &json_data);
+
   // Send the data to the renderer.
-  scoped_ptr<Value> data_values(value);
-  web_ui()->CallJavascriptFunction(
-      "g_browserBridge.receivedData", *data_values.get());
+  web_ui()->CallJavascriptFunction("g_browserBridge.receivedData", json_data);
 }
-