Site Settings Desktop: Implement USB devices section.

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 22 matching lines @@
  *            exceptions: ExceptionListPref}}
  */
 var SiteSettingsPref;
 
 /**
  * @typedef {{name: string,
  *            id: string}}
  */
 var MediaPickerEntry;
 
+/**
+ * @typedef {{name: string,
+ *            product-id: Number,
+ *            serial-number: string,
+ *            vendor-id: Number}}
+ */
+var UsbDeviceDetails;
+
+/**
+ * @typedef {{embeddingOrigin: string,
+ *            object: UsbDeviceDetails,
+ *            objectName: string,
+ *            origin: string,
+ *            setting: string,
+ *            source: string}}
+ */
+var UsbDeviceEntry;
+
 cr.define('settings', function() {
   /** @interface */
   function SiteSettingsPrefsBrowserProxy() {}
 
   SiteSettingsPrefsBrowserProxy.prototype = {
     /**
      * Sets the default value for a site settings category.
      * @param {string} contentType The name of the category to change.
      * @param {string} defaultValue The name of the value to set as default.
      */
@@ skipping 94 matching lines @@
      * @param {string} url The url to use as the default.
      */
     setProtocolDefault: function(protocol, url) {},
 
     /**
      * Deletes a certain protocol handler by url.
      * @param {string} protocol The protocol to delete the url from.
      * @param {string} url The url to delete.
      */
     removeProtocolHandler: function(protocol, url) {},
+
+    /**
+     * Fetches a list of all USB devices and the sites permitted to use them.
+     * @return {!Promise<Array<UsbDeviceEntry>>} The list of USB devices.
+     */
+    fetchUsbDevices: function() {},
+
+    /**
+     * Removes a particular USB device object permission by origin and embedding
+     * origin.
+     * @param {string} origin The origin to look up the permission for.
+     * @param {string} embeddingOrigin the embedding origin to look up.
+     * @param {UsbDeviceDetails} object The USB device object to revoke

[dschuyler, 2016/08/11 23:28:07]

nit: how about the name |usbDevice| instead of |object| for the parameter?

[Finnur, 2016/08/12 13:11:26]

I like that idea.

+     *     permission for.
+     */
+    removeUsbDevice: function(origin, embeddingOrigin, object) {},
   };
 
   /**
    * @constructor
    * @implements {settings.SiteSettingsPrefsBrowserProxy}
    */
   function SiteSettingsPrefsBrowserProxyImpl() {}
 
   // The singleton instance_ is replaced with a test version of this wrapper
   // during testing.
@@ skipping 70 matching lines @@
 
     /** @override */
     setProtocolDefault: function(protocol, url) {
       chrome.send('setDefault', [[protocol, url]]);
     },
 
     /** @override */
     removeProtocolHandler: function(protocol, url) {
       chrome.send('removeHandler', [[protocol, url]]);
     },
+
+    /** @override */
+    fetchUsbDevices: function() {
+      return cr.sendWithPromise('fetchUsbDevices');
+    },
+
+    /** @override */
+    removeUsbDevice: function(origin, embeddingOrigin, object) {

[dschuyler, 2016/08/11 23:28:07]

Nit: is there another param name we can use other than |object|
it's too generic imo.

[Finnur, 2016/08/12 13:11:26]

Done.

+      chrome.send('removeUsbDevice', [origin, embeddingOrigin, object]);
+    },
   };
 
   return {
     SiteSettingsPrefsBrowserProxy: SiteSettingsPrefsBrowserProxy,
     SiteSettingsPrefsBrowserProxyImpl: SiteSettingsPrefsBrowserProxyImpl,
   };
 });