Log operator code histogram on new metric log

chrome/browser/chrome_browser_main.cc

Index: chrome/browser/chrome_browser_main.cc
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index 19c4cadab5fe4ddf3b108aa51e81dfdd12490bea..a8e8119ce8d6aa65a02a620c87e39cdb082fa912 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -42,6 +42,7 @@
 #include "chrome/browser/browser_process_platform_part.h"
 #include "chrome/browser/browser_shutdown.h"
 #include "chrome/browser/chrome_browser_main_extra_parts.h"
+#include "chrome/browser/chrome_browser_metrics_log_observer.h"
 #include "chrome/browser/component_updater/cld_component_installer.h"
 #include "chrome/browser/component_updater/component_updater_service.h"
 #include "chrome/browser/component_updater/flash_component_installer.h"
@@ -559,9 +560,11 @@ void ChromeBrowserMainParts::SetupMetricsAndFieldTrials() {
   MetricsLog::set_version_extension("-64");
 #endif  // defined(ARCH_CPU_64_BITS)
 
+  MetricsService* metrics = browser_process_->metrics_service();

[bengr, 2014/05/06 17:14:39]

Why did you move this line up? And should the comment move with it?

[bolian, 2014/05/06 18:00:08]

Move it so that the metrics service could be created before adding the log
observer. The comment is for the lines below.

+  // Create the metrics log observer.
+  browser_metrics_log_observer_.reset(new ChromeBrowserMetricsLogObserver());
   // Initialize FieldTrialList to support FieldTrials that use one-time
   // randomization.
-  MetricsService* metrics = browser_process_->metrics_service();
   MetricsService::ReportingState reporting_state =
       IsMetricsReportingEnabled() ? MetricsService::REPORTING_ENABLED :
                                     MetricsService::REPORTING_DISABLED;