Index: chrome/browser/ui/webui/sync_promo/sync_promo_handler2.cc |
diff --git a/chrome/browser/ui/webui/sync_promo/sync_promo_handler2.cc b/chrome/browser/ui/webui/sync_promo/sync_promo_handler2.cc |
index 6e96dabf2449fc5a4379596c0d2b6b65b2a225e9..275b6071ebbcd0702ae92205fdb4ab1dab309c07 100644 |
--- a/chrome/browser/ui/webui/sync_promo/sync_promo_handler2.cc |
+++ b/chrome/browser/ui/webui/sync_promo/sync_promo_handler2.cc |
@@ -205,7 +205,7 @@ void SyncPromoHandler2::HandleCloseSyncPromo(const base::ListValue* args) { |
// close. |
Browser* browser = |
BrowserList::FindBrowserWithWebContents(web_ui()->GetWebContents()); |
- if (browser && !browser->IsAttemptingToCloseBrowser()) { |
+ if (|browser || !browser->IsAttemptingToCloseBrowser()) { |
Dan Beam
2012/01/30 18:10:05
s/|/!/
sail
2012/01/30 21:52:36
Done. Ouch, good catch.
|
GURL url = SyncPromoUI::GetNextPageURLForSyncPromoURL( |
web_ui_->tab_contents()->GetURL()); |
web_ui_->tab_contents()->OpenURL(url, GURL(), CURRENT_TAB, |
@@ -213,8 +213,27 @@ void SyncPromoHandler2::HandleCloseSyncPromo(const base::ListValue* args) { |
} |
} |
+int SyncPromoHandler2::GetPromoVersion() { |
+ switch (SyncPromoUI::GetSyncPromoVersion()) { |
+ case SyncPromoUI::VERSION_DEFAULT: |
+ return 0; |
+ case SyncPromoUI::VERSION_DEVICES: |
+ return 1; |
+ case SyncPromoUI::VERSION_VERBOSE: |
+ return 2; |
+ case SyncPromoUI::VERSION_SIMPLE: |
+ return 3; |
+ case SyncPromoUI::VERSION_DIALOG: |
+ // Use the simple sync promo layout for the dialog version. |
+ return 3; |
+ default: |
+ NOTREACHED(); |
+ return 0; |
+ } |
+} |
+ |
void SyncPromoHandler2::HandleInitializeSyncPromo(const base::ListValue* args) { |
- base::FundamentalValue version(SyncPromoUI::GetSyncPromoVersion()); |
+ base::FundamentalValue version(GetPromoVersion()); |
web_ui_->CallJavascriptFunction("SyncSetupOverlay.showPromoVersion", |
version); |