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

Unified Diff: chrome/browser/resources/options2/search_page.js

Issue 9464053: Hide/Disable several (meaningless) options in Settings uber-page for Guest. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: No change picture button anymore Created 8 years, 10 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/options2/search_page.js
diff --git a/chrome/browser/resources/options2/search_page.js b/chrome/browser/resources/options2/search_page.js
index caaa647a1b67bad22882726dcc47f17c366e55e0..010e69ac80f3250dc719e890d284b2b945d798d3 100644
--- a/chrome/browser/resources/options2/search_page.js
+++ b/chrome/browser/resources/options2/search_page.js
@@ -312,7 +312,9 @@ cr.define('options', function() {
page = pagesToSearch[key];
var elements = page.pageDiv.querySelectorAll('section');
for (var i = 0, node; node = elements[i]; i++) {
- node.classList.add('search-hidden');
+ if (!node.hidden) {
James Hawkins 2012/02/28 20:03:35 No braces.
Denis Kuznetsov (DE-MUC) 2012/02/29 08:41:43 Done.
+ node.classList.add('search-hidden');
+ }
}
}
@@ -325,9 +327,11 @@ cr.define('options', function() {
var elements =
page.pageDiv.querySelectorAll('section');
for (var i = 0, node; node = elements[i]; i++) {
- if (this.performReplace_(regEx, replaceString, node)) {
- node.classList.remove('search-hidden');
- foundMatches = true;
+ if (!node.hidden) {
+ if (this.performReplace_(regEx, replaceString, node)) {
+ node.classList.remove('search-hidden');
+ foundMatches = true;
+ }
}
}
}

Powered by Google App Engine
This is Rietveld 408576698