Store USB device permissions in website settings.

chrome/browser/permissions/chooser_permission_context.h

Index: chrome/browser/permissions/chooser_permission_context.h
diff --git a/chrome/browser/permissions/permission_context.h b/chrome/browser/permissions/chooser_permission_context.h
similarity index 54%
copy from chrome/browser/permissions/permission_context.h
copy to chrome/browser/permissions/chooser_permission_context.h
index 7dfd84edbac55f7bc66889b4c728e3cccbae7097..875e0663d185e855b55693377d5335c67481afed 100644
--- a/chrome/browser/permissions/permission_context.h
+++ b/chrome/browser/permissions/chooser_permission_context.h
@@ -2,15 +2,15 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_H_
-#define CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_H_
+#ifndef CHROME_BROWSER_PERMISSIONS_CHOOSER_PERMISSION_CONTEXT_H_
+#define CHROME_BROWSER_PERMISSIONS_CHOOSER_PERMISSION_CONTEXT_H_
 
 #include <list>
 
 #include "base/macros.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 
-class PermissionContextBase;
+class ChooserPermissionContextBase;
 class Profile;
 class KeyedServiceBaseFactory;
 
@@ -18,24 +18,24 @@ namespace content {
 enum class PermissionType;
 };  // namespace content
 
-class PermissionContext {
+class ChooserPermissionContext {

[raymes, 2015/10/14 02:07:04]

Can we avoid adding this entire class for the time being and just access the the
UsbChooserPermissionContext directly? It looks like it's not used in this patch
anyway atm.

[Reilly Grant (use Gerrit), 2015/10/22 00:57:02]

It's used in my next patch and it makes more sense to me to introduce it here
with the rest of the ChooserPermissionContext infrastructure than later in a
primarily UI-focused patch.

[raymes, 2015/10/29 06:14:40]

I don't feel entirely convinced that this is needed though - but it's hard to
see becaues nothing in this CL uses it :) Since this CL is large, could we carve
this off (along with the enum) and add it in a separate CL?

[Reilly Grant (use Gerrit), 2015/10/29 20:59:59]

Done.

  public:
-  // Helper method returning the PermissionContextBase object associated with
-  // the given ContentSettingsType.
+  // Helper method returning the ChooserPermissionContextBase object associated
+  // with the given ContentSettingsType.
   // This can return nullptr if the the permission type has no associated
   // context.
-  static PermissionContextBase* Get(
+  static ChooserPermissionContextBase* Get(
       Profile* profile,
       content::PermissionType content_settings_type);
 
-  // Return all the factories related to PermissionContext. These are the
-  // factories used by ::Get() to create a PermissionContextBase.
+  // Return all the factories related to ChooserPermissionContext. These are the
+  // factories used by ::Get() to create a ChooserPermissionContextBase.
   // This is meant to be used by callers of ::Get() that need to depend on these
   // factories.
   static const std::list<KeyedServiceBaseFactory*>& GetFactories();
 
  private:
-  DISALLOW_IMPLICIT_CONSTRUCTORS(PermissionContext);
+  DISALLOW_IMPLICIT_CONSTRUCTORS(ChooserPermissionContext);
 };
 
-#endif // CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_H_
+#endif  // CHROME_BROWSER_PERMISSIONS_CHOOSER_PERMISSION_CONTEXT_H_