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

Unified Diff: chrome/browser/resources/options/search_engine_manager_engine_list.js

Issue 11189088: Add controlled setting indicator for default search engine configuration (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comment addressed. Created 8 years, 2 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/options/search_engine_manager_engine_list.js
diff --git a/chrome/browser/resources/options/search_engine_manager_engine_list.js b/chrome/browser/resources/options/search_engine_manager_engine_list.js
index ae9e5a8a669a6e6ad916eb3f8b4fb0c0cf188d33..d84bfe7ab1edf8e12f2397e1ab97790aab2c5a5b 100644
--- a/chrome/browser/resources/options/search_engine_manager_engine_list.js
+++ b/chrome/browser/resources/options/search_engine_manager_engine_list.js
@@ -3,6 +3,8 @@
// found in the LICENSE file.
cr.define('options.search_engines', function() {
+ /** @const */ var ControlledSettingIndicator =
+ options.ControlledSettingIndicator;
/** @const */ var InlineEditableItemList = options.InlineEditableItemList;
/** @const */ var InlineEditableItem = options.InlineEditableItem;
/** @const */ var ListSelectionController = cr.ui.ListSelectionController;
@@ -168,6 +170,15 @@ cr.define('options.search_engines', function() {
this.addEventListener('commitedit', this.onEditCommitted_.bind(this));
} else {
this.editable = false;
+ this.querySelector('.row-delete-button').hidden = true;
+ var indicator = ControlledSettingIndicator();
+ indicator.setAttribute('setting', 'search-engine');
+ // Create a synthetic pref change event decorated as
+ // CoreOptionsHandler::CreateValueForPref() does.
+ var event = new cr.Event(this.contentType);
+ event.value = { controlledBy: 'policy' };
+ indicator.handlePrefChange(event);
+ this.appendChild(indicator);
}
},
« no previous file with comments | « chrome/browser/resources/options/browser_options.js ('k') | chrome/browser/ui/webui/options/browser_options_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698