Small cleanup for ChromeBrowserMain:

chrome/browser/chrome_browser_main_linux.cc

Index: chrome/browser/chrome_browser_main_linux.cc
===================================================================
--- chrome/browser/chrome_browser_main_linux.cc	(revision 210624)
+++ chrome/browser/chrome_browser_main_linux.cc	(working copy)
@@ -16,6 +16,8 @@
 #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"
@@ -133,6 +135,17 @@
   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();