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

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

Issue 2686063004: MD Settings: make blowing away per-origin data (i.e. cookies) easier (Closed)
Patch Set: fix test Created 3 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/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 83f02809fa6e3c89e6a478123b6d1c9651161df5..82fd3e8c27cab1ea645ac7e492304d079fd12c2f 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
@@ -52,6 +52,8 @@ var TestSiteSettingsPrefsBrowserProxy = function() {
'getExceptionList',
'observeProtocolHandlers',
'observeProtocolHandlersEnabledState',
+ 'reloadCookies',
+ 'removeCookie',
'removeProtocolHandler',
'removeUsbDevice',
'removeZoomLevel',
@@ -228,6 +230,16 @@ TestSiteSettingsPrefsBrowserProxy.prototype = {
},
/** @override */
+ reloadCookies: function() {
+ return Promise.resolve({id: null, children: []});
dschuyler 2017/02/13 20:14:35 Would it make sense to define a private var around
Dan Beam 2017/02/14 04:14:04 not until the setter would be used, no
+ },
+
+ /** @override */
+ removeCookie: function(path) {
+ this.methodCalled('removeCookie', path);
+ },
+
+ /** @override */
removeZoomLevel: function(host) {
this.methodCalled('removeZoomLevel', [host]);
},

Powered by Google App Engine
This is Rietveld 408576698