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

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

Issue 7669052: Added DataTypeController integration and UI surfacing for syncing Search Engines. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Fixed notification registration. Created 9 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/sync_setup_overlay.js
===================================================================
--- chrome/browser/resources/sync_setup_overlay.js (revision 98647)
+++ chrome/browser/resources/sync_setup_overlay.js (working copy)
@@ -256,6 +256,7 @@
"syncExtensions": syncAll || $('extensions-checkbox').checked,
"syncTypedUrls": syncAll || $('typed-urls-checkbox').checked,
"syncApps": syncAll || $('apps-checkbox').checked,
+ "syncSearchEngines": syncAll || $('search-engines-checkbox').checked,
"syncSessions": syncAll || $('sessions-checkbox').checked,
"encryptAllData": encryptAllData,
"usePassphrase": usePassphrase,
@@ -349,14 +350,20 @@
} else {
$('apps-item').className = "sync-item-hide";
}
-
- this.setCheckboxesToKeepEverythingSynced_(args.keepEverythingSynced);
+ if (args.searchEnginesRegistered) {
+ $('search-engines-checkbox').checked = args.syncSearchEngines;
+ $('search-engines-item').className = "sync-item-show";
+ } else {
+ $('search-engines-item').className = "sync-item-hide";
+ }
if (args.sessionsRegistered) {
$('sessions-checkbox').checked = args.syncSessions;
$('sessions-item').className = "sync-item-show";
} else {
$('sessions-item').className = "sync-item-hide";
}
+
+ this.setCheckboxesToKeepEverythingSynced_(args.keepEverythingSynced);
},
setEncryptionRadios_: function(args) {
« no previous file with comments | « chrome/browser/resources/sync_setup_overlay.html ('k') | chrome/browser/search_engines/template_url_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698