Small cleanup for ChromeBrowserMain:

chrome/browser/chrome_browser_main.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROME_BROWSER_MAIN_H_
 #define CHROME_BROWSER_CHROME_BROWSER_MAIN_H_
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/metrics/field_trial.h"
@@ skipping 83 matching lines @@
   virtual void ShowMissingLocaleMessageBox() = 0;
 
   const content::MainFunctionParams& parameters() const {
     return parameters_;
   }
   const CommandLine& parsed_command_line() const {
     return parsed_command_line_;
   }
 
   Profile* profile() { return profile_; }
+  bool do_first_run_tasks() const { return do_first_run_tasks_; }
 
   const PrefService* local_state() const { return local_state_; }
 
  private:
   // Methods for |SetupMetricsAndFieldTrials()| --------------------------------
 
   // Constructs metrics service and does related initialization, including
   // creation of field trials. Call only after labs have been converted to
   // switches.
   void SetupMetricsAndFieldTrials();
 
   // Starts recording of metrics. This can only be called after we have a file
   // thread.
   void StartMetricsRecording();
 
   // Returns true if the user opted in to sending metric reports.
   bool IsMetricsReportingEnabled();
 
+  // Record time from process startup to present time in an UMA histogram.
+  // |is_first_run| - is the current launch part of a first run.
+  void RecordBrowserStartupTime(bool is_first_run);
+
+  // Records a time value to an UMA histogram in the context of the
+  // PreReadExperiment field-trial. This also reports to the appropriate
+  // sub-histogram (_PreRead(Enabled|Disabled)).
+  void RecordPreReadExperimentTime(const char* name, base::TimeDelta time);
+
   // Methods for Main Message Loop -------------------------------------------
 
   int PreCreateThreadsImpl();
   int PreMainMessageLoopRunImpl();
 
   // Members initialized on construction ---------------------------------------
 
   const content::MainFunctionParams& parameters_;
   const CommandLine& parsed_command_line_;
   int result_code_;
@@ skipping 35 matching lines @@
       tracking_synchronizer_;
 #if !defined(OS_ANDROID)
   // Browser creation happens on the Java side in Android.
   scoped_ptr<StartupBrowserCreator> browser_creator_;
 
   // Android doesn't support multiple browser processes, so it doesn't implement
   // ProcessSingleton.
   scoped_ptr<ChromeProcessSingleton> process_singleton_;
 #endif
   scoped_ptr<first_run::MasterPrefs> master_prefs_;
-  bool record_search_engine_;
   TranslateManager* translate_manager_;
   Profile* profile_;
   bool run_message_loop_;
   ProcessSingleton::NotifyResult notify_result_;
   scoped_ptr<ThreeDAPIObserver> three_d_observer_;
 
   // Initialized in SetupMetricsAndFieldTrials.
   scoped_refptr<FieldTrialSynchronizer> field_trial_synchronizer_;
 
   // Members initialized in PreMainMessageLoopRun, needed in
   // PreMainMessageLoopRunThreadsCreated.
   bool do_first_run_tasks_;
   PrefService* local_state_;
   base::FilePath user_data_dir_;
 
 #if !defined(OS_ANDROID)
   scoped_ptr<ActiveTabTracker> active_tab_tracker_;
 #endif
 
   // Members needed across shutdown methods.
   bool restart_last_session_;
 
   // Tests can set this to true to disable restricting cookie access in the
   // network stack, as this can only be done once.
   static bool disable_enforcing_cookie_policies_for_tests_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeBrowserMainParts);
 };
 
-// Records the conditions that can prevent Breakpad from generating and
-// sending crash reports.  The presence of a Breakpad handler (after
-// attempting to initialize crash reporting) and the presence of a debugger
-// are registered with the UMA metrics service.
-void RecordBreakpadStatusUMA(MetricsService* metrics);
-
-// Displays a warning message if some minimum level of OS support is not
-// present on the current platform.
-void WarnAboutMinimumSystemRequirements();
-
-// Record time from process startup to present time in an UMA histogram.
-// |is_first_run| - is the current launch part of a first run.
-void RecordBrowserStartupTime(bool is_first_run);
-
-// Records a time value to an UMA histogram in the context of the
-// PreReadExperiment field-trial. This also reports to the appropriate
-// sub-histogram (_PreRead(Enabled|Disabled)).
-void RecordPreReadExperimentTime(const char* name, base::TimeDelta time);
-
 #endif  // CHROME_BROWSER_CHROME_BROWSER_MAIN_H_