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

Unified Diff: chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js

Issue 2598893002: [MD settings] lighter weight initializer for protocol handler enabled messages (Closed)
Patch Set: review changes Created 4 years 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
« no previous file with comments | « chrome/test/data/webui/settings/protocol_handlers_tests.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js
diff --git a/chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js b/chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js
index a0ef8dcfb2b39b8521082569b4ef138385144345..46d3bfa7d967b60e6fb2799a3834465d6bdb94a6 100644
--- a/chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js
+++ b/chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js
@@ -50,7 +50,8 @@ var TestSiteSettingsPrefsBrowserProxy = function() {
'fetchZoomLevels',
'getDefaultValueForContentType',
'getExceptionList',
- 'initializeProtocolHandlerList',
+ 'observeProtocolHandlers',
+ 'observeProtocolHandlersEnabledState',
'removeProtocolHandler',
'removeUsbDevice',
'removeZoomLevel',
@@ -241,10 +242,16 @@ TestSiteSettingsPrefsBrowserProxy.prototype = {
},
/** @override */
- initializeProtocolHandlerList: function() {
+ observeProtocolHandlers: function() {
cr.webUIListenerCallback('setHandlersEnabled', true);
cr.webUIListenerCallback('setProtocolHandlers', this.protocolHandlers_);
- this.methodCalled('initializeProtocolHandlerList');
+ this.methodCalled('observeProtocolHandlers');
+ },
+
+ /** @override */
+ observeProtocolHandlersEnabledState: function() {
+ cr.webUIListenerCallback('setHandlersEnabled', true);
+ this.methodCalled('observeProtocolHandlersEnabledState');
},
/** @override */
« no previous file with comments | « chrome/test/data/webui/settings/protocol_handlers_tests.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698