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

Unified Diff: chrome/browser/resources/sync_setup_overlay.js

Issue 11445002: Sync user's custom spellcheck dictionary (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Add browser tests for dictionary change notifications in settings Created 8 years 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/sync_setup_overlay.js
diff --git a/chrome/browser/resources/sync_setup_overlay.js b/chrome/browser/resources/sync_setup_overlay.js
index 57957d0b447365437cea70d894f624e2eecef4cb..7191c988d6e331e02d7128b785a21ac3cf9025a6 100644
--- a/chrome/browser/resources/sync_setup_overlay.js
+++ b/chrome/browser/resources/sync_setup_overlay.js
@@ -270,6 +270,7 @@ cr.define('options', function() {
'typedUrlsSynced': syncAll || $('typed-urls-checkbox').checked,
'appsSynced': syncAll || $('apps-checkbox').checked,
'sessionsSynced': syncAll || $('sessions-checkbox').checked,
+ 'dictionarySynced': syncAll || $('dictionary-checkbox').checked,
'encryptAllData': encryptAllData,
'usePassphrase': usePassphrase,
'isGooglePassphrase': googlePassphrase,
@@ -376,6 +377,12 @@ cr.define('options', function() {
} else {
$('sessions-item').hidden = true;
}
+ if (args.dictionaryRegistered) {
+ $('dictionary-checkbox').checked = args.dictionarySynced;
+ $('dictionary-item').hidden = false;
+ } else {
+ $('dictionary-item').hidden = true;
+ }
this.setCheckboxesToKeepEverythingSynced_(args.syncAllDataTypes);
},

Powered by Google App Engine
This is Rietveld 408576698