Small cleanup for ChromeBrowserMain:

chrome/browser/chrome_browser_main_linux.cc

 // Copyright (c) 2012 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/chrome_browser_main_linux.h"
 
 #if !defined(OS_CHROMEOS)
 #include "chrome/browser/storage_monitor/storage_monitor_linux.h"
 #include "content/public/browser/browser_thread.h"
 #endif
 
 #if defined(USE_LINUX_BREAKPAD)
 #include <stdlib.h>
 
 #include "base/command_line.h"
 #include "base/linux_util.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/app/breakpad_linux.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/env_vars.h"
 #include "chrome/common/pref_names.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/chromeos/settings/cros_settings_names.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chromeos/chromeos_switches.h"
 #endif
@@ skipping 97 matching lines @@
 #endif
 
 #if !defined(OS_CHROMEOS)
   const base::FilePath kDefaultMtabPath("/etc/mtab");
   storage_monitor_.reset(new chrome::StorageMonitorLinux(kDefaultMtabPath));
 #endif
 
   ChromeBrowserMainPartsPosix::PreProfileInit();
 }
 
+void ChromeBrowserMainPartsLinux::PostProfileInit() {
+  ChromeBrowserMainPartsPosix::PostProfileInit();
+
+#if defined(USE_LINUX_BREAKPAD)
+  g_browser_process->metrics_service()->RecordBreakpadRegistration(
+      IsCrashReporterEnabled());
+#else
+  g_browser_process->metrics_service()->RecordBreakpadRegistration(false);

[petarj, 2013/07/11 23:31:04]

You can not use g_browser_process variable here, since it is not
defined (chrome/browser/browser_process.h is not included).
I.e. this breaks the build if USE_LINUX_BREAKPAD is not defined.

Can you please fix this?

Thank you.

[jam, 2013/07/12 00:06:05]

derat has fixed this

+#endif

[Nico, 2013/07/10 17:50:52]

(and this)

+}
+
 void ChromeBrowserMainPartsLinux::PostMainMessageLoopRun() {
   ChromeBrowserMainPartsPosix::PostMainMessageLoopRun();
 
 #if !defined(OS_CHROMEOS)
   // Delete it now. Otherwise the FILE thread would be gone when we try to
   // release it in the dtor and Valgrind would report a leak on almost every
   // single browser_test.
   storage_monitor_.reset();
 #endif
 }