Show sync promo at startup

chrome/browser/profiles/profile_manager.cc

Index: chrome/browser/profiles/profile_manager.cc
diff --git a/chrome/browser/profiles/profile_manager.cc b/chrome/browser/profiles/profile_manager.cc
index 3586a41177351111f92cca0b7de1969584932c70..b3dd87beca8f12a88c7162b5ac32d8b54cc36e4c 100644
--- a/chrome/browser/profiles/profile_manager.cc
+++ b/chrome/browser/profiles/profile_manager.cc
@@ -20,6 +20,7 @@
 #include "chrome/browser/sessions/session_service_factory.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/ui/browser_window.h"
+#include "chrome/browser/ui/webui/sync_promo_ui.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
@@ -66,8 +67,14 @@ class NewProfileLauncher : public ProfileManagerObserver {
     if (status == STATUS_INITIALIZED) {
       DCHECK(profile);
       Browser* browser = Browser::Create(profile);
-      browser->AddSelectedTabWithURL(GURL(chrome::kChromeUINewTabURL),
-                                     PageTransition::LINK);
+      GURL url;

[sky, 2011/10/01 20:32:42]

This doesn't seem right. Shouldn't we be calling into browser_init here?
Otherwise we're not doing session restore and other random things we might want
to trigger.

[sail, 2011/10/01 21:37:34]

This code is only called if the user adds a new profile by clicking the "Add
profile" button. At this point we'd never want to do session restore or open the
welcome page here.

Are you sure we should still be calling browser_init from here?

+      if (SyncPromoUI::ShouldShowSyncPromoAtStartup(profile, true)) {
+        SyncPromoUI::DidShowSyncPromoAtStartup(profile);
+        url = GURL(chrome::kChromeUISyncPromoURL);
+      } else {
+        url = GURL(chrome::kChromeUINewTabURL);
+      }
+      browser->AddSelectedTabWithURL(url, PageTransition::LINK);
       browser->window()->Show();
     }
   }