Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Side by Side Diff: chrome/browser/permissions/chooser_permission_context.h

Issue 1382783002: Store USB device permissions in website settings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove UsbPermissionContext and add ChooserPermissionContext helper functions. Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_H_ 5 #ifndef CHROME_BROWSER_PERMISSIONS_CHOOSER_PERMISSION_CONTEXT_H_
6 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_H_ 6 #define CHROME_BROWSER_PERMISSIONS_CHOOSER_PERMISSION_CONTEXT_H_
7 7
8 #include <list> 8 #include <list>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "components/content_settings/core/common/content_settings_types.h" 11 #include "components/content_settings/core/common/content_settings_types.h"
12 12
13 class PermissionContextBase; 13 class ChooserPermissionContextBase;
14 class Profile; 14 class Profile;
15 class KeyedServiceBaseFactory; 15 class KeyedServiceBaseFactory;
16 16
17 namespace content { 17 namespace content {
18 enum class PermissionType; 18 enum class PermissionType;
19 }; // namespace content 19 }; // namespace content
20 20
21 class PermissionContext { 21 class ChooserPermissionContext {
raymes 2015/10/14 02:07:04 Can we avoid adding this entire class for the time
Reilly Grant (use Gerrit) 2015/10/22 00:57:02 It's used in my next patch and it makes more sense
raymes 2015/10/29 06:14:40 I don't feel entirely convinced that this is neede
Reilly Grant (use Gerrit) 2015/10/29 20:59:59 Done.
22 public: 22 public:
23 // Helper method returning the PermissionContextBase object associated with 23 // Helper method returning the ChooserPermissionContextBase object associated
24 // the given ContentSettingsType. 24 // with the given ContentSettingsType.
25 // This can return nullptr if the the permission type has no associated 25 // This can return nullptr if the the permission type has no associated
26 // context. 26 // context.
27 static PermissionContextBase* Get( 27 static ChooserPermissionContextBase* Get(
28 Profile* profile, 28 Profile* profile,
29 content::PermissionType content_settings_type); 29 content::PermissionType content_settings_type);
30 30
31 // Return all the factories related to PermissionContext. These are the 31 // Return all the factories related to ChooserPermissionContext. These are the
32 // factories used by ::Get() to create a PermissionContextBase. 32 // factories used by ::Get() to create a ChooserPermissionContextBase.
33 // This is meant to be used by callers of ::Get() that need to depend on these 33 // This is meant to be used by callers of ::Get() that need to depend on these
34 // factories. 34 // factories.
35 static const std::list<KeyedServiceBaseFactory*>& GetFactories(); 35 static const std::list<KeyedServiceBaseFactory*>& GetFactories();
36 36
37 private: 37 private:
38 DISALLOW_IMPLICIT_CONSTRUCTORS(PermissionContext); 38 DISALLOW_IMPLICIT_CONSTRUCTORS(ChooserPermissionContext);
39 }; 39 };
40 40
41 #endif // CHROME_BROWSER_PERMISSIONS_PERMISSION_CONTEXT_H_ 41 #endif // CHROME_BROWSER_PERMISSIONS_CHOOSER_PERMISSION_CONTEXT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698