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

Unified Diff: chrome/browser/sync/resources/configure.html

Issue 7669052: Added DataTypeController integration and UI surfacing for syncing Search Engines. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Initial upload. 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/sync/resources/configure.html
===================================================================
--- chrome/browser/sync/resources/configure.html (revision 97454)
+++ chrome/browser/sync/resources/configure.html (working copy)
@@ -359,16 +359,22 @@
args.syncApps;
document.getElementById("appsItem").className = "sync-item-show";
} else {
- document.getElementById("appsItem").className = "sync-item-hide";
- }
-
+ document.getElementById("appsItem").className = "sync-item-hide";
+ }
+ if (args.searchEnginesRegistered) {
+ document.getElementById("searchEnginesCheckbox").checked = args.syncSearchEngines;
Nicolas Zea 2011/08/19 18:32:39 line length
SteveT 2011/08/22 18:33:42 Done.
+ document.getElementById("searchEnginesItem").className = "sync-item-show";
+ } else {
+ document.getElementById("searchEnginesItem").className = "sync-item-hide";
+ }
+ if (args.sessionsRegistered) {
+ document.getElementById("sessionsCheckbox").checked = args.syncSessions;
+ document.getElementById("sessionsItem").className = "sync-item-show";
+ } else {
+ document.getElementById("sessionsItem").className = "sync-item-hide";
+ }
+
setCheckboxesToKeepEverythingSynced(args.keepEverythingSynced);
- if (args.sessionsRegistered) {
- document.getElementById("sessionsCheckbox").checked = args.syncSessions;
- document.getElementById("sessionsItem").className = "sync-item-show";
- } else {
- document.getElementById("sessionsItem").className = "sync-item-hide";
- }
}
function setEncryptionCheckboxes(args) {
@@ -455,8 +461,9 @@
"syncPasswords": syncAll || f.passwordsCheckbox.checked,
"syncAutofill": syncAll || f.autofillCheckbox.checked,
"syncExtensions": syncAll || f.extensionsCheckbox.checked,
- "syncTypedUrls": syncAll || f.typedUrlsCheckbox.checked,
- "syncApps": syncAll || f.appsCheckbox.checked,
+ "syncTypedUrls": syncAll || f.typedUrlsCheckbox.checked,
+ "syncApps": syncAll || f.appsCheckbox.checked,
+ "syncSearchEngines": syncAll || f.searchEnginesCheckbox.checked,
"syncSessions": syncAll || f.sessionsCheckbox.checked,
"usePassphrase": (getRadioCheckedValue() == 'explicit'),
"passphrase": f.passphrase.value
@@ -612,6 +619,13 @@
for="sessionsCheckbox" i18n-content="foreignsessions"
il8n-values="title:sessions"></label>
</div>
+ <!-- Search Engines -->
+ <div class="sync-item-show" id="searchEnginesItem">
+ <input id="searchEnginesCheckbox" name="dataTypeCheckbox" type="checkbox">
+ <label id="searchEnginesCheckboxLabel" name="dataTypeLabel"
+ for="searchEnginesCheckbox" i18n-content="searchengines"
+ il8n-values="title:searchengines"></label>
+ </div>
</div>
</div>
</div>

Powered by Google App Engine
This is Rietveld 408576698