MD Settings: extension-controlled indicators for default search engine

chrome/browser/resources/settings/search_page/search_page.js

Index: chrome/browser/resources/settings/search_page/search_page.js
diff --git a/chrome/browser/resources/settings/search_page/search_page.js b/chrome/browser/resources/settings/search_page/search_page.js
index 4c70a30433cf313549cede562266954b265e04a7..328df5bd0351d400cb89a6eada3e760923e42281 100644
--- a/chrome/browser/resources/settings/search_page/search_page.js
+++ b/chrome/browser/resources/settings/search_page/search_page.js
@@ -10,6 +10,8 @@ Polymer({
   is: 'settings-search-page',
 
   properties: {
+    prefs: Object,
+
     /**
      * List of default search engines available.
      * @private {!Array<!SearchEngine>}
@@ -18,11 +20,11 @@ Polymer({
       type: Array,
       value: function() { return []; }
     },
-
-    /** @private {!settings.SearchEnginesBrowserProxy} */
-    browserProxy_: Object,
   },
 
+  /** @private {?settings.SearchEnginesBrowserProxy} */
+  browserProxy_: null,
+
   /** @override */
   created: function() {
     this.browserProxy_ = settings.SearchEnginesBrowserProxyImpl.getInstance();
@@ -38,14 +40,19 @@ Polymer({
   },
 
   /** @private */
-  onManageSearchEnginesTap_: function() {
-    settings.navigateTo(settings.Route.SEARCH_ENGINES);
-  },
-
-  /** @private */
-  onChange_: function(e) {
+  onChange_: function() {
     var select = /** @type {!HTMLSelectElement} */ (this.$$('select'));
     var searchEngine = this.searchEngines_[select.selectedIndex];
     this.browserProxy_.setDefaultSearchEngine(searchEngine.modelIndex);
   },
+
+  /** @private */
+  onDisableExtension_: function() {
+    this.fire('refresh-pref', 'default_search_provider.enabled');
+  },
+
+  /** @private */
+  onManageSearchEnginesTap_: function() {

[dpapad, 2016/11/22 18:35:50]

Does the "manage search engines" UI also need to be updated when the default
search engine is controlled by an extension? Asking because that UI also allows
changing the default search engine.

[Dan Beam, 2016/11/23 02:15:14]

yes :)

+    settings.navigateTo(settings.Route.SEARCH_ENGINES);
+  },
 });