diff options
| author | mirandac@chromium.org <mirandac@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-21 21:43:17 +0000 |
|---|---|---|
| committer | mirandac@chromium.org <mirandac@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-21 21:43:17 +0000 |
| commit | 17ba1120adc5deae49bf6100f430498501b0c3a8 (patch) | |
| tree | 4b17a6890965b827ff66179abca64b2c3c6f0f5c /chrome | |
| parent | d7833abb0a4de4cd2cfd73da7bb375e51928a77f (diff) | |
| download | chromium_src-17ba1120adc5deae49bf6100f430498501b0c3a8.zip chromium_src-17ba1120adc5deae49bf6100f430498501b0c3a8.tar.gz chromium_src-17ba1120adc5deae49bf6100f430498501b0c3a8.tar.bz2 | |
Remove old bookmark sync promo from new tab page.
BUG= 28050, 27338
TEST= old bookmark sync promo should no longer appear on NTP.
Review URL: http://codereview.chromium.org/420004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32754 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/app/generated_resources.grd | 6 | ||||
| -rw-r--r-- | chrome/browser/dom_ui/new_tab_page_sync_handler.cc | 10 | ||||
| -rw-r--r-- | chrome/browser/dom_ui/new_tab_page_sync_handler.h | 1 |
3 files changed, 1 insertions, 16 deletions
diff --git a/chrome/app/generated_resources.grd b/chrome/app/generated_resources.grd index f7e0da7..2efa902 100644 --- a/chrome/app/generated_resources.grd +++ b/chrome/app/generated_resources.grd @@ -5604,12 +5604,6 @@ Keep your key file in a safe place. You will need it to create new versions of y <message name="IDS_SYNC_NTP_SYNC_SECTION_ERROR_TITLE" desc="The title to display in the New Tab Page sync status section in case of an error."> Bookmark Sync Error! </message> - <message name="IDS_SYNC_NTP_PROMOTION_MESSAGE" desc="The New Tab Page sync promotion message to display in the sync status section."> - You can sync your bookmarks across computers using your Google account - </message> - <message name="IDS_SYNC_NTP_START_NOW_LINK_LABEL" desc="The label to display for the New Tab Page promotion link."> - Start now - </message> <message name="IDS_SYNC_NTP_SETUP_IN_PROGRESS" desc="The message to display in the New Tab Page sync section when the sync setup is underway."> Setup in progress... </message> diff --git a/chrome/browser/dom_ui/new_tab_page_sync_handler.cc b/chrome/browser/dom_ui/new_tab_page_sync_handler.cc index 6773756..466931a 100644 --- a/chrome/browser/dom_ui/new_tab_page_sync_handler.cc +++ b/chrome/browser/dom_ui/new_tab_page_sync_handler.cc @@ -125,15 +125,8 @@ void NewTabPageSyncHandler::BuildAndSendSyncStatus() { return; } - // We show the sync promotion if sync has not been enabled and the user is - // logged in to Google Accounts. If the user is not signed in to GA, we - // should hide the sync status section entirely. + // Don't show sync status if setup is not complete. if (!sync_service_->HasSyncSetupCompleted()) { - if (!sync_service_->SetupInProgress() && IsGoogleGAIACookieInstalled()) { - SendSyncMessageToPage(PROMOTION, - WideToUTF8(l10n_util::GetString(IDS_SYNC_NTP_PROMOTION_MESSAGE)), - WideToUTF8(l10n_util::GetString(IDS_SYNC_NTP_START_NOW_LINK_LABEL))); - } return; } @@ -182,7 +175,6 @@ void NewTabPageSyncHandler::SendSyncMessageToPage( std::string linkurl; switch (type) { case HIDE: - case PROMOTION: msgtype = "presynced"; break; case SYNC_ERROR: diff --git a/chrome/browser/dom_ui/new_tab_page_sync_handler.h b/chrome/browser/dom_ui/new_tab_page_sync_handler.h index cde61da..67b565c 100644 --- a/chrome/browser/dom_ui/new_tab_page_sync_handler.h +++ b/chrome/browser/dom_ui/new_tab_page_sync_handler.h @@ -36,7 +36,6 @@ class NewTabPageSyncHandler : public DOMMessageHandler, private: enum MessageType { HIDE, - PROMOTION, SYNC_ERROR, }; // Helper to invoke the |syncMessageChanged| JS function on the new tab page. |