Reporting top cpu and memory consumers via rappor on chromeos

chrome/browser/chromeos/chrome_browser_main_chromeos.cc

Index: chrome/browser/chromeos/chrome_browser_main_chromeos.cc
diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
index 08bc59e3a02b9e7958d2e0e11147f04513f0d721..f55506e17feaeee92d8f73af525b931d4417ec55 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
@@ -65,6 +65,7 @@
 #include "chrome/browser/chromeos/power/power_prefs.h"
 #include "chrome/browser/chromeos/power/renderer_freezer.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
+#include "chrome/browser/chromeos/resource_reporter/resource_reporter.h"
 #include "chrome/browser/chromeos/settings/device_oauth2_token_service_factory.h"
 #include "chrome/browser/chromeos/settings/device_settings_service.h"
 #include "chrome/browser/chromeos/status/data_promo_notification.h"
@@ -400,6 +401,8 @@ void ChromeBrowserMainPartsChromeos::PreMainMessageLoopRun() {
 #endif
 
   ChromeBrowserMainPartsLinux::PreMainMessageLoopRun();
+
+  chromeos::ResourceReporter::GetInstance()->StartMonitoring();

[Daniel Erat, 2015/11/23 23:57:19]

it seems a bit weird to me that everything else in this message gets run before
ChromeBrowserMainPartsLinux::PreMainMessageLoopRun(), but this runs after. if
that's intentional and not a bug, please add a comment here explaining why.

[afakhry, 2015/11/24 01:52:32]

This is an arbitrary decision. It doesn't matter if it's before or after.

 }
 
 void ChromeBrowserMainPartsChromeos::PreProfileInit() {
@@ -710,6 +713,8 @@ void ChromeBrowserMainPartsChromeos::PostBrowserStart() {
 
 // Shut down services before the browser process, etc are destroyed.
 void ChromeBrowserMainPartsChromeos::PostMainMessageLoopRun() {
+  chromeos::ResourceReporter::GetInstance()->StopMonitoring();
+
   BootTimesRecorder::Get()->AddLogoutTimeMarker("UIMessageLoopEnded", true);
 
 #if defined(ENABLE_ARC)