diff options
author | rsimha@chromium.org <rsimha@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-29 01:55:21 +0000 |
---|---|---|
committer | rsimha@chromium.org <rsimha@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-29 01:55:21 +0000 |
commit | 588ad3c98ab23f0f0945d14fefd594fccdec38b7 (patch) | |
tree | 9cfee07ac100f572e53e57cdf5e8895eeaecac95 /chrome/browser/sync/sync_ui_util.cc | |
parent | 9b4cd1b5eff9385649c0f4eb781f513ed82bb8e7 (diff) | |
download | chromium_src-588ad3c98ab23f0f0945d14fefd594fccdec38b7.zip chromium_src-588ad3c98ab23f0f0945d14fefd594fccdec38b7.tar.gz chromium_src-588ad3c98ab23f0f0945d14fefd594fccdec38b7.tar.bz2 |
[sync] Hide "Sync nothing" UI; Go back to signing out on dashboard clears and abandoned initial setup
A new "Sync nothing" option was recently added to the advanced sync
settings dialog. It allowed the user to stop syncing, but stay signed
in. In addition, sign in and sync were separated, with the result that
on a dashboard clear, the user would remain signed in, but would no
longer have sync enabled.
For M29, we've decided to hide this UI and revert to old sign-in/out
behavior because there are other services that depend on the sync
engine being up and running.
This patch undoes a small number of these changes as listed below:
1) Hides the "Sync nothing" item in the advanced sync settings drop-down
2) Signs out the user if the user clicks "Choose what to sync" during
initial setup, and then clicks cancel on the advanced dialog
3) Signs out the user when a dashboard clear is done
4) Signs out the user if the user clicks cancel while re-setting up sync
after the dashboard clear
5) Reverts the status string shown when sync is cleared via the dashboard
BUG=235633, 252049, 248877
TEST=No "Sync nothing" UI in advanced settings; user is signed out when sync is disabled on Desktop; Chrome OS strings back to M28
Review URL: https://chromiumcodereview.appspot.com/17392009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209267 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/sync/sync_ui_util.cc')
-rw-r--r-- | chrome/browser/sync/sync_ui_util.cc | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/chrome/browser/sync/sync_ui_util.cc b/chrome/browser/sync/sync_ui_util.cc index 46aebd5..645bc4c 100644 --- a/chrome/browser/sync/sync_ui_util.cc +++ b/chrome/browser/sync/sync_ui_util.cc @@ -56,10 +56,8 @@ string16 GetSyncedStateStatusLabel(ProfileSyncService* service, user_name); } else if (service->IsStartSuppressed()) { // User is signed in, but sync has been stopped. - return l10n_util::GetStringFUTF16( - IDS_SIGNED_IN_WITH_SYNC_SUPPRESSED, - user_name, - ASCIIToUTF16(chrome::kSyncGoogleDashboardURL)); + return l10n_util::GetStringFUTF16(IDS_SIGNED_IN_WITH_SYNC_SUPPRESSED, + user_name); } } @@ -253,8 +251,7 @@ MessageType GetStatusInfo(ProfileSyncService* service, if (status_label) { string16 label = l10n_util::GetStringFUTF16( IDS_SIGNED_IN_WITH_SYNC_SUPPRESSED, - UTF8ToUTF16(signin.GetAuthenticatedUsername()), - ASCIIToUTF16(chrome::kSyncGoogleDashboardURL)); + UTF8ToUTF16(signin.GetAuthenticatedUsername())); status_label->assign(label); result_type = PRE_SYNCED; } |