MD Settings: Amend SiteSettingsHandlerTest.Origins to test "Site Details" code.

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview A helper object used from the "Site Settings" section to
  * interact with the content settings prefs.
  */
 
 /**
@@ skipping 132 matching lines @@
      * 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) {}
 
     /**
+     * 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) {}

[dschuyler, 2017/07/05 18:30:18]

It looks like this is being added a second time (maybe a merge whoops).

[Patti Lor, 2017/07/06 04:23:40]

Oops, thanks for picking this up! Fixed.

+
+    /**
      * Sets the category permission for a given origin (expressed as primary
      *    and secondary patterns).
      * @param {string} primaryPattern The origin to change (primary pattern).
      * @param {string} secondaryPattern The embedding origin to change
      *    (secondary pattern).
      * @param {string} contentType The name of the category to change.
      * @param {string} value The value to change the permission to.
      * @param {boolean} incognito Whether this rule applies only to the current
      *     incognito session.
      */
@@ skipping 260 matching lines @@
 
   // The singleton instance_ is replaced with a test version of this wrapper
   // during testing.
   cr.addSingletonGetter(SiteSettingsPrefsBrowserProxyImpl);
 
   return {
     SiteSettingsPrefsBrowserProxy: SiteSettingsPrefsBrowserProxy,
     SiteSettingsPrefsBrowserProxyImpl: SiteSettingsPrefsBrowserProxyImpl,
   };
 });