Store USB device permissions in website settings.

chrome/browser/permissions/chooser_context_base.h

Index: chrome/browser/permissions/chooser_context_base.h
diff --git a/chrome/browser/permissions/chooser_context_base.h b/chrome/browser/permissions/chooser_context_base.h
new file mode 100644
index 0000000000000000000000000000000000000000..2d984b0a7e2781b241695860c0e51059c01d6676
--- /dev/null
+++ b/chrome/browser/permissions/chooser_context_base.h
@@ -0,0 +1,76 @@
+// Copyright 2015 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.
+
+#ifndef CHROME_BROWSER_PERMISSIONS_CHOOSER_CONTEXT_BASE_H_
+#define CHROME_BROWSER_PERMISSIONS_CHOOSER_CONTEXT_BASE_H_
+
+#include <string>
+
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/scoped_vector.h"
+#include "components/content_settings/core/common/content_settings_types.h"
+#include "components/keyed_service/core/keyed_service.h"
+
+namespace base {
+class DictionaryValue;
+class Value;
+}
+
+class GURL;
+class HostContentSettingsMap;
+class Profile;
+
+// This is the base class for services that manage any type of permission that
+// is granted through a chooser-style UI instead of a simple allow/deny prompt.
+// Subclasses must define the structure of the objects that are stored.
+class ChooserContextBase : public KeyedService {
+ public:
+  ChooserContextBase(Profile* profile,
+                     ContentSettingsType permission_type,
+                     ContentSettingsType chooser_type);
+  ~ChooserContextBase() override;
+
+  // Returns the list of objects that |requesting_origin| has been granted
+  // permission to access when embedded within |embedding_origin|.
+  //
+  // This method may be extended by a subclass to return objects not stored in
+  // |host_content_settings_map_|.
+  virtual ScopedVector<base::DictionaryValue> GetPreviouslyChosenObjects(

[raymes, 2015/11/02 03:00:13]

This and the function below weren't virtual in the previous patchset :) I have
some thoughts on this but I think it will be better for you to land this without
further changes. Are you ok to make these non-virtual again and make those
changes in a separate CL? If so then lgtm :)

[Reilly Grant (use Gerrit), 2015/11/02 21:14:51]

Done.

+      const GURL& requesting_origin,
+      const GURL& embedding_origin);
+
+  // Grants |requesting_origin| access to |object| when embedded within
+  // |embedding_origin| by writing it into |host_content_settings_map_|.
+  void GrantObjectPermission(const GURL& requesting_origin,
+                             const GURL& embedding_origin,
+                             scoped_ptr<base::DictionaryValue> object);
+
+  // Revokes |requesting_origin|'s permission to access |object| when embedded
+  // within |embedding_origin|.
+  //
+  // This method may be extended by a subclass to be revoke permission for
+  // objects returned by GetPreviouslyChosenObjects but not stored in
+  // |host_content_settings_map_|.
+  virtual void RevokeObjectPermission(const GURL& requesting_origin,
+                                      const GURL& embedding_origin,
+                                      const base::DictionaryValue& object);
+
+  // Validates the structure of an object read from
+  // |host_content_settings_map_|.
+  virtual bool IsValidObject(const base::DictionaryValue& object) = 0;
+
+ private:
+  scoped_ptr<base::DictionaryValue> GetWebsiteSetting(
+      const GURL& requesting_origin,
+      const GURL& embedding_origin);
+  void SetWebsiteSetting(const GURL& requesting_origin,
+                         const GURL& embedding_origin,
+                         scoped_ptr<base::Value> value);
+
+  HostContentSettingsMap* const host_content_settings_map_;
+  const ContentSettingsType permission_type_;
+  const ContentSettingsType chooser_type_;
+};
+
+#endif  // CHROME_BROWSER_PERMISSIONS_CHOOSER_CONTEXT_BASE_H_