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

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

Issue 2699013002: MD Settings: Allow editing a cookie site exception. (Closed)
Patch Set: Address nits. 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
« no previous file with comments | « chrome/test/data/webui/settings/site_list_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 dd9abed76a8fa03899e486cbb22933541f303d73..6fe9b1ca4a670e19f9e30d280c432135acc73109 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
@@ -53,6 +53,7 @@ var TestSiteSettingsPrefsBrowserProxy = function() {
'getCookieDetails',
'getDefaultValueForContentType',
'getExceptionList',
+ 'isPatternValid',
'observeProtocolHandlers',
'observeProtocolHandlersEnabledState',
'reloadCookies',
@@ -80,6 +81,9 @@ var TestSiteSettingsPrefsBrowserProxy = function() {
/** @private {?CookieList} */
this.cookieDetails_ = null;
+
+ /** @private {boolean} */
+ this.isPatternValid_ = true;
};
TestSiteSettingsPrefsBrowserProxy.prototype = {
@@ -231,6 +235,19 @@ TestSiteSettingsPrefsBrowserProxy.prototype = {
},
/** @override */
+ isPatternValid: function(pattern) {
+ this.methodCalled('isPatternValid', pattern);
+ return Promise.resolve(this.isPatternValid_);
+ },
+
+ /**
+ * Specify whether isPatternValid should succeed or fail.
+ */
+ setIsPatternValid: function(isValid) {
+ this.isPatternValid_ = isValid;
+ },
+
+ /** @override */
resetCategoryPermissionForOrigin: function(
primaryPattern, secondaryPattern, contentType, incognito) {
this.methodCalled('resetCategoryPermissionForOrigin',
@@ -242,7 +259,7 @@ TestSiteSettingsPrefsBrowserProxy.prototype = {
setCategoryPermissionForOrigin: function(
primaryPattern, secondaryPattern, contentType, value, incognito) {
this.methodCalled('setCategoryPermissionForOrigin',
- [primaryPattern, secondaryPattern, contentType, value]);
+ [primaryPattern, secondaryPattern, contentType, value, incognito]);
return Promise.resolve();
},
« no previous file with comments | « chrome/test/data/webui/settings/site_list_tests.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698