Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(591)

Side by Side Diff: chrome/browser/resources/settings/people_page/sync_browser_proxy.js

Issue 2920853004: [sync] Display an error when sync settings aren't confirmed (Closed)
Patch Set: initialize button to nullptr Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview A helper object used from the the People section to get the 6 * @fileoverview A helper object used from the the People section to get the
7 * status of the sync backend and user preferences on what data to sync. Used 7 * status of the sync backend and user preferences on what data to sync. Used
8 * for both Chrome browser and ChromeOS. 8 * for both Chrome browser and ChromeOS.
9 */ 9 */
10 cr.exportPath('settings'); 10 cr.exportPath('settings');
(...skipping 22 matching lines...) Expand all
33 * Must be kept in sync with the return values of getSyncErrorAction in 33 * Must be kept in sync with the return values of getSyncErrorAction in
34 * chrome/browser/ui/webui/settings/people_handler.cc 34 * chrome/browser/ui/webui/settings/people_handler.cc
35 * @enum {string} 35 * @enum {string}
36 */ 36 */
37 settings.StatusAction = { 37 settings.StatusAction = {
38 NO_ACTION: 'noAction', // No action to take. 38 NO_ACTION: 'noAction', // No action to take.
39 REAUTHENTICATE: 'reauthenticate', // User needs to reauthenticate. 39 REAUTHENTICATE: 'reauthenticate', // User needs to reauthenticate.
40 SIGNOUT_AND_SIGNIN: 'signOutAndSignIn', // User needs to sign out and sign in. 40 SIGNOUT_AND_SIGNIN: 'signOutAndSignIn', // User needs to sign out and sign in.
41 UPGRADE_CLIENT: 'upgradeClient', // User needs to upgrade the client. 41 UPGRADE_CLIENT: 'upgradeClient', // User needs to upgrade the client.
42 ENTER_PASSPHRASE: 'enterPassphrase', // User needs to enter passphrase. 42 ENTER_PASSPHRASE: 'enterPassphrase', // User needs to enter passphrase.
43 CONFIRM_SYNC_SETTINGS: "confirmSyncSettings",
tommycli 2017/06/12 18:41:01 nit: We use single quotes in JS for strings. Unfor
Patrick Noland 2017/06/12 18:44:30 Done.
44 // User needs to confirm sync settings.
43 }; 45 };
44 46
45 /** 47 /**
46 * The state of sync. This is the data structure sent back and forth between 48 * The state of sync. This is the data structure sent back and forth between
47 * C++ and JS. Its naming and structure is not optimal, but changing it would 49 * C++ and JS. Its naming and structure is not optimal, but changing it would
48 * require changes to the C++ handler, which is already functional. 50 * require changes to the C++ handler, which is already functional.
49 * @typedef {{ 51 * @typedef {{
50 * appsEnforced: boolean, 52 * appsEnforced: boolean,
51 * appsRegistered: boolean, 53 * appsRegistered: boolean,
52 * appsSynced: boolean, 54 * appsSynced: boolean,
(...skipping 180 matching lines...) Expand 10 before | Expand all | Expand 10 after
233 chrome.metricsPrivate.recordUserAction( 235 chrome.metricsPrivate.recordUserAction(
234 'Signin_AccountSettings_GoogleActivityControlsClicked'); 236 'Signin_AccountSettings_GoogleActivityControlsClicked');
235 } 237 }
236 }; 238 };
237 239
238 return { 240 return {
239 SyncBrowserProxy: SyncBrowserProxy, 241 SyncBrowserProxy: SyncBrowserProxy,
240 SyncBrowserProxyImpl: SyncBrowserProxyImpl, 242 SyncBrowserProxyImpl: SyncBrowserProxyImpl,
241 }; 243 };
242 }); 244 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/settings/people_page/people_page.js ('k') | chrome/browser/sync/sync_ui_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698