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

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

Issue 2936003003: MD Settings: Set all content setting values in Site Details Javascript. (Closed)
Patch Set: Review comments. Created 3 years, 6 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/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 acd83ba4f33c6f12905da0fce9dd40c8a8815435..a72a963bc052a11d63962489069b7cb2da9e2965 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
@@ -19,7 +19,7 @@ var ContentSettingProvider = {
};
/**
- * The site exception information passed form the C++ handler.
+ * The site exception information passed from the C++ handler.
* See also: SiteException.
* @typedef {{embeddingOrigin: string,
* incognito: boolean,
@@ -126,13 +126,6 @@ cr.define('settings', function() {
getExceptionList(contentType) {}
/**
- * Gets the exception details for a particular site.
- * @param {string} site The name of the site.
- * @return {!Promise<!RawSiteException>}
- */
- getSiteDetails(site) {}
-
- /**
* Resets the category permission for a given origin (expressed as primary
* and secondary patterns).
* @param {string} primaryPattern The origin to change (primary pattern).
@@ -146,6 +139,17 @@ cr.define('settings', function() {
primaryPattern, secondaryPattern, contentType, incognito) {}
/**
+ * Gets a list of category permissions for a given origin. Note that this
+ * may be different to the results retrieved by getExceptionList(), since it
+ * combines different sources of data to get a permission's value.
+ * @param {string} origin The origin to look up permissions for.
+ * @param {!Array<string>} contentTypes A list of categories to retrieve
+ * the ContentSetting for.
+ * @return {!Promise<!NodeList<!RawSiteException>>}
+ */
+ getOriginPermissions(origin, contentTypes) {}
+
+ /**
* Sets the category permission for a given origin (expressed as primary
* and secondary patterns).
* @param {string} primaryPattern The origin to change (primary pattern).
@@ -308,11 +312,6 @@ cr.define('settings', function() {
}
/** @override */
- getSiteDetails(site) {
- return cr.sendWithPromise('getSiteDetails', site);
- }
-
- /** @override */
resetCategoryPermissionForOrigin(
primaryPattern, secondaryPattern, contentType, incognito) {
chrome.send(
@@ -321,6 +320,11 @@ cr.define('settings', function() {
}
/** @override */
+ getOriginPermissions(origin, contentTypes) {
+ return cr.sendWithPromise('getOriginPermissions', origin, contentTypes);
+ }
+
+ /** @override */
setCategoryPermissionForOrigin(
primaryPattern, secondaryPattern, contentType, value, incognito) {
// TODO(dschuyler): It may be incorrect for JS to send the embeddingOrigin

Powered by Google App Engine
This is Rietveld 408576698