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

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

Issue 2552883009: [MD settings] include source of policy for content settings (Closed)
Patch Set: account for unset source 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
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 9c9345878edaf114f73ac85cd677f6d0d3d1ca22..b50e5369c1600258171553562ff9225e4a67b78c 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
@@ -8,6 +8,17 @@
*/
/**
+ * The handler will send a police source that is similar, but not exactly the
+ * same as a ControlledBy value. If the ContentSettingProvider is omitted it
+ * should be treated as 'default'.
+ * @enum {string}
+ */
+var ContentSettingProvider = {
+ EXTENSION: 'extension',
+ PREFERENCE: 'preference',
+};
+
+/**
* @typedef {{embeddingOrigin: string,
* embeddingDisplayName: string,
* incognito: boolean,
@@ -25,6 +36,12 @@ var SiteException;
var CategoryDefaultsPref;
/**
+ * @typedef {{setting: string,
+ * source: ContentSettingProvider}}
+ */
+var DefaultContentSetting;
+
+/**
* @typedef {{location: Array<SiteException>,
* notifications: Array<SiteException>}}
*/
@@ -96,7 +113,7 @@ cr.define('settings', function() {
/**
* 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<!DefaultContentSetting>}
*/
getDefaultValueForContentType: function(contentType) {},

Powered by Google App Engine
This is Rietveld 408576698