[MD setting] separate RawSiteException and SiteException

chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js

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 651257799ea56235bdc92ae0301a28b2e7da055e..21537218f65c497033ac673f9beabd1752bceb51 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,6 +19,8 @@ var ContentSettingProvider = {
 };
 
 /**
+ * The site exception information passed form the C++ handler.
+ * See also: SiteException.
  * @typedef {{embeddingOrigin: string,
  *            embeddingDisplayName: string,
  *            incognito: boolean,
@@ -27,6 +29,20 @@ var ContentSettingProvider = {
  *            setting: string,
  *            source: string}}
  */
+var RawSiteException;
+
+/**
+ * The site exception after it has been converted/filtered for UI use.
+ * See also: RawSiteException.
+ * @typedef {{category: string,

[dpapad, 2017/03/21 01:04:22]

Follow up question from previous comment.

1) Should this be
category: settings.ContentSettingsTypes
2) Should this type be moved to site_settings/constants.js? It is not used
anywhere in the proxy anymore (see related following comment too).

[dschuyler, 2017/03/22 01:35:41]

It can be. Done.
That's a reasonable idea. I'd like to keep this Cl relatively small since it's
just a cleanup/enhancement.

+ *            embeddingOrigin: string,
+ *            embeddingDisplayName: string,
+ *            incognito: boolean,
+ *            origin: string,
+ *            displayName: string,
+ *            setting: string,
+ *            source: string}}
+ */
 var SiteException;
 
 /**
@@ -120,14 +136,14 @@ cr.define('settings', function() {
     /**
      * 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<!RawSiteException>>}
      */
     getExceptionList: function(contentType) {},
 
     /**
      * Gets the exception details for a particular site.
      * @param {string} site The name of the site.
-     * @return {!Promise<!SiteException>}
+     * @return {!Promise<!RawSiteException>}
      */
     getSiteDetails: function(site) {},