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..a9392ceeb89763726d7ec60a3faab55258d212f0 100644
--- a/chrome/browser/ui/browser_init.h
+++ b/chrome/browser/ui/browser_init.h
@@ -64,9 +64,10 @@ 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.
+  // |is_first_run| indicates that this is a new profile.
   bool LaunchBrowser(const CommandLine& command_line, Profile* profile,
                      const FilePath& cur_dir, bool process_startup,
-                     int* return_code);
+                     bool is_first_run, int* return_code);

[sky, 2011/10/04 15:33:53]

Each param on its own line, here and all other methods.
Also, multiple booleans like this make the code nearly unreadable. Use an enum
(style guide encourages this too).

[sail, 2011/10/04 19:25:22]

Done.

 
   // LaunchWithProfile ---------------------------------------------------------
   //
@@ -97,10 +98,11 @@ 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);
+    // the first run.  |is_first_run| indicates that this is a new profile.
     LaunchWithProfile(const FilePath& cur_dir, const CommandLine& command_line,
-                      BrowserInit* browser_init);
+                      bool is_first_run);
+    LaunchWithProfile(const FilePath& cur_dir, const CommandLine& command_line,
+                      BrowserInit* browser_init, bool is_first_run);
     ~LaunchWithProfile();
 
     // Creates the necessary windows for startup. Returns true on success,
@@ -213,6 +215,7 @@ class BrowserInit {
     const CommandLine& command_line_;
     Profile* profile_;
     BrowserInit* browser_init_;
+    bool is_first_run_;
     DISALLOW_COPY_AND_ASSIGN(LaunchWithProfile);
   };