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

Unified Diff: chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js

Issue 2476753002: [MD settings] get cookie details from site without transfering cookie tree (Closed)
Patch Set: closure fix Created 4 years, 1 month 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/settings/site_settings/site_settings_prefs_browser_proxy.js
diff --git a/chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js b/chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js
index e25039d32518d1034a8f99005fbccbbe02e7f67e..02b9ca2212ba0798c9bcd7d370ac678f350d2fac 100644
--- a/chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js
+++ b/chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js
@@ -87,23 +87,30 @@ cr.define('settings', function() {
setDefaultValueForContentType: function(contentType, defaultValue) {},
/**
+ * Gets the cookie details for a particular site.
+ * @param {string} site The name of the site.
+ * @return {!Promise<!CookieDataSummaryItem>}
+ */
+ getCookieDetails: function(site) {},
+
+ /**
* Gets the default value for a site settings category.
* @param {string} contentType The name of the category to query.
- * @return {Promise<string>} The string value of the default setting.
+ * @return {!Promise<string>} The string value of the default setting.
*/
getDefaultValueForContentType: function(contentType) {},
/**
* Gets the exceptions (site list) for a particular category.
* @param {string} contentType The name of the category to query.
- * @return {Promise<Array<SiteException>>}
+ * @return {!Promise<!Array<!SiteException>>}
*/
getExceptionList: function(contentType) {},
/**
* Gets the exception details for a particular site.
* @param {string} site The name of the site.
- * @return {Promise<SiteException>}
+ * @return {!Promise<!SiteException>}
*/
getSiteDetails: function(site) {},
@@ -221,7 +228,7 @@ cr.define('settings', function() {
* origin.
* @param {string} origin The origin to look up the permission for.
* @param {string} embeddingOrigin the embedding origin to look up.
- * @param {UsbDeviceDetails} usbDevice The USB device to revoke permission
+ * @param {!UsbDeviceDetails} usbDevice The USB device to revoke permission
* for.
*/
removeUsbDevice: function(origin, embeddingOrigin, usbDevice) {},
@@ -262,6 +269,11 @@ cr.define('settings', function() {
},
/** @override */
+ getCookieDetails: function(site) {
+ return cr.sendWithPromise('getCookieDetails', site);
+ },
+
+ /** @override */
getDefaultValueForContentType: function(contentType) {
return cr.sendWithPromise('getDefaultValueForContentType', contentType);
},

Powered by Google App Engine
This is Rietveld 408576698