Show sync promo at startup

chrome/browser/ui/browser_init.h

Index: chrome/browser/ui/browser_init.h
diff --git a/chrome/browser/ui/browser_init.h b/chrome/browser/ui/browser_init.h
index 24466ea0e2d13c12f5f31765d42c55d5669de65e..876327cc7da19ccbd0fdc41d7297d3b59d4604c6 100644
--- a/chrome/browser/ui/browser_init.h
+++ b/chrome/browser/ui/browser_init.h
@@ -24,6 +24,15 @@ class TabContentsWrapper;
 // initialize the profile.
 class BrowserInit {
  public:
+  enum IsProcessStartup {
+    IS_NOT_PROCESS_STARTUP,
+    IS_PROCESS_STARTUP
+  };
+  enum IsFirstRun {
+    IS_NOT_FIRST_RUN,
+    IS_FIRST_RUN
+  };
+
   BrowserInit();
   ~BrowserInit();
 
@@ -64,8 +73,12 @@ class BrowserInit {
   // be the command line passed to this process. |cur_dir| can be empty, which
   // implies that the directory of the executable should be used.
   // |process_startup| indicates whether this is the first browser.
-  bool LaunchBrowser(const CommandLine& command_line, Profile* profile,
-                     const FilePath& cur_dir, bool process_startup,
+  // |is_first_run| indicates that this is a new profile.
+  bool LaunchBrowser(const CommandLine& command_line,
+                     Profile* profile,
+                     const FilePath& cur_dir,
+                     IsProcessStartup is_process_startup,
+                     IsFirstRun is_first_run,
                      int* return_code);
 
   // LaunchWithProfile ---------------------------------------------------------
@@ -97,10 +110,14 @@ class BrowserInit {
     // There are two ctors. The first one implies a NULL browser_init object
     // and thus no access to distribution-specific first-run behaviors. The
     // second one is always called when the browser starts even if it is not
-    // the first run.
-    LaunchWithProfile(const FilePath& cur_dir, const CommandLine& command_line);
-    LaunchWithProfile(const FilePath& cur_dir, const CommandLine& command_line,
-                      BrowserInit* browser_init);
+    // the first run.  |is_first_run| indicates that this is a new profile.
+    LaunchWithProfile(const FilePath& cur_dir,
+                      const CommandLine& command_line,
+                      IsFirstRun is_first_run);
+    LaunchWithProfile(const FilePath& cur_dir,
+                      const CommandLine& command_line,
+                      BrowserInit* browser_init,
+                      IsFirstRun is_first_run);
     ~LaunchWithProfile();
 
     // Creates the necessary windows for startup. Returns true on success,
@@ -213,6 +230,7 @@ class BrowserInit {
     const CommandLine& command_line_;
     Profile* profile_;
     BrowserInit* browser_init_;
+    IsFirstRun is_first_run_;

[sky, 2011/10/04 20:13:23]

Feel free to use a boolean here. The enum is meant to clarify the call site, but
when you're in code like this is_first_run_ is better served by a boolean.

[sail, 2011/10/04 20:56:24]

Done.

     DISALLOW_COPY_AND_ASSIGN(LaunchWithProfile);
   };