Make GTK and Aura parts orthogonal to OS parts

chrome/browser/chrome_browser_main.h

 // Copyright (c) 2011 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_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
+#include "base/memory/scoped_vector.h"
 #include "base/metrics/field_trial.h"
 #include "base/tracked_objects.h"
 #include "chrome/browser/first_run/first_run.h"
 #include "chrome/browser/process_singleton.h"
 #include "content/browser/browser_main.h"
 
 class BrowserProcessImpl;
 class FieldTrialSynchronizer;
 class HistogramSynchronizer;
 class MetricsService;
 class PrefService;
 class Profile;
 class ShutdownWatcherHelper;
 class TranslateManager;
 
 namespace chrome_browser {
 // For use by ShowMissingLocaleMessageBox.
 extern const char kMissingLocaleDataTitle[];
 extern const char kMissingLocaleDataMessage[];
 }
 
 class ChromeBrowserMainParts : public content::BrowserMainParts {
  public:
   virtual ~ChromeBrowserMainParts();
 
+  // Subclass to handle setup for parts orthagonal to the OS (GTK, Aura, etc).
+  class ExtraParts {
+   public:
+    explicit ExtraParts(ChromeBrowserMainParts* main_parts);
+    virtual ~ExtraParts();
+
+    virtual void PreEarlyInitialization();
+    virtual void PostEarlyInitialization();
+    virtual void PreMainMessageLoopRun();
+    virtual void PostMainMessageLoopRun();
+
+   protected:
+    ChromeBrowserMainParts* main_parts() { return main_parts_; }
+
+   private:
+    ChromeBrowserMainParts* main_parts_;
+
+    DISALLOW_COPY_AND_ASSIGN(ExtraParts);
+  };
+
   // Constructs metrics service and does related initialization, including
   // creation of field trials. Call only after labs have been converted to
   // switches.
   MetricsService* SetupMetricsAndFieldTrials(
       const CommandLine& parsed_command_line,
       PrefService* local_state);
 
  protected:
   explicit ChromeBrowserMainParts(const MainFunctionParams& parameters);
 
+  // content::BrowserMainParts overrides
+  virtual void PreEarlyInitialization() OVERRIDE;
+  virtual void PostEarlyInitialization() OVERRIDE;
   virtual void PreMainMessageLoopRun() OVERRIDE;
-  int PreMainMessageLoopRunInternal();
   virtual void MainMessageLoopRun() OVERRIDE;
   virtual void PostMainMessageLoopRun() OVERRIDE;
   virtual void ToolkitInitialized() OVERRIDE;
 
+  // New virtuals
+
+  // Displays a warning message that we can't find any locale data files.
+  virtual void ShowMissingLocaleMessageBox() = 0;
+  // Handles the --hide-icons and --show-icons command line options.
+  virtual int HandleIconsCommands() = 0;
+  // Check if there is any machine level Chrome installed. If so, handle it
+  // and return true.
+  virtual bool CheckMachineLevelInstall() = 0;
+  // Prepare localization, etc for handling crashes.
+  virtual void PrepareRestartOnCrashEnviroment() = 0;
+
  private:
+  int PreMainMessageLoopRunInternal();
+
   // Methods for |EarlyInitialization()| ---------------------------------------
 
   // A/B test for the maximum number of persistent connections per host.
   void ConnectionFieldTrial();
 
   // A/B test for determining a value for unused socket timeout.
   void SocketTimeoutFieldTrial();
 
   // A/B test for the maximum number of connections per proxy server.
   void ProxyConnectionsFieldTrial();
@@ skipping 37 matching lines @@
   // Creating this object starts tracking the creation and deletion of Task
   // instance. This MUST be done before main_message_loop, so that it is
   // destroyed after the main_message_loop.
   tracked_objects::AutoTracking tracking_objects_;
 #endif
 
   // Statistical testing infrastructure for the entire browser. NULL until
   // SetupMetricsAndFieldTrials is called.
   scoped_ptr<base::FieldTrialList> field_trial_list_;
 
+  // Vector of extra parts instantiated in the constructor.
+  // The virtual methods on each of these are called in the order added.
+  // They are releaed (destroyed) in the opposite order.
+  typedef ScopedVector<ExtraParts> PartsList;
+  PartsList extra_parts_;
+
   // Members initialized after / released before main_message_loop_ ------------
 
   scoped_ptr<BrowserProcessImpl> browser_process_;
   scoped_refptr<HistogramSynchronizer> histogram_synchronizer_;
   scoped_ptr<ProcessSingleton> process_singleton_;
   scoped_ptr<FirstRun::MasterPrefs> master_prefs_;
   bool record_search_engine_;
   TranslateManager* translate_manager_;
   Profile* profile_;
   bool run_message_loop_;
@@ skipping 11 matching lines @@
 // 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();
 
-// Displays a warning message that we can't find any locale data files.
-void ShowMissingLocaleMessageBox();
-
 // Records the time from our process' startup to the present time in
 // the UMA histogram |metric_name|.
 void RecordBrowserStartupTime();
 
 // 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_