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

Side by Side Diff: chrome/common/extensions/features/base_feature_provider.h

Issue 11316164: Implement ComplexFeature to support permission features with multiple rules. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Renames and add TODO Created 8 years 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_COMMON_EXTENSIONS_FEATURES_SIMPLE_FEATURE_PROVIDER_H_ 5 #ifndef CHROME_COMMON_EXTENSIONS_FEATURES_BASE_FEATURE_PROVIDER_H_
6 #define CHROME_COMMON_EXTENSIONS_FEATURES_SIMPLE_FEATURE_PROVIDER_H_ 6 #define CHROME_COMMON_EXTENSIONS_FEATURES_BASE_FEATURE_PROVIDER_H_
7 7
8 #include <map>
8 #include <set> 9 #include <set>
9 #include <string> 10 #include <string>
10 11
11 #include "base/memory/linked_ptr.h" 12 #include "base/memory/linked_ptr.h"
12 #include "base/values.h" 13 #include "base/values.h"
13 #include "chrome/common/extensions/features/feature.h"
14 #include "chrome/common/extensions/features/feature_provider.h" 14 #include "chrome/common/extensions/features/feature_provider.h"
15 #include "chrome/common/extensions/features/simple_feature.h"
15 16
16 namespace extensions { 17 namespace extensions {
17 18
18 // Reads Features out of a simple JSON file description. 19 // Reads Features out of a simple JSON file description.
19 class SimpleFeatureProvider : public FeatureProvider { 20 class BaseFeatureProvider : public FeatureProvider {
20 public: 21 public:
21 typedef Feature*(*FeatureFactory)(); 22 typedef SimpleFeature*(*FeatureFactory)();
22 23
23 // Creates a new SimpleFeatureProvider. Pass null to |factory| to have the 24 // Creates a new BaseFeatureProvider. Pass null to |factory| to have the
24 // provider create plain old Feature instances. 25 // provider create plain old Feature instances.
25 SimpleFeatureProvider(DictionaryValue* root, FeatureFactory factory); 26 BaseFeatureProvider(const DictionaryValue& root, FeatureFactory factory);
26 virtual ~SimpleFeatureProvider(); 27 virtual ~BaseFeatureProvider();
27 28
28 // Gets an instance for the _manifest_features.json file that is baked into 29 // Gets an instance for the _manifest_features.json file that is baked into
29 // Chrome as a resource. 30 // Chrome as a resource.
30 static SimpleFeatureProvider* GetManifestFeatures(); 31 static BaseFeatureProvider* GetManifestFeatures();
31 32
32 // Gets an instance for the _permission_features.json file that is baked into 33 // Gets an instance for the _permission_features.json file that is baked into
33 // Chrome as a resource. 34 // Chrome as a resource.
34 static SimpleFeatureProvider* GetPermissionFeatures(); 35 static BaseFeatureProvider* GetPermissionFeatures();
35 36
36 // Returns all features described by this instance. 37 // Returns all features described by this instance.
37 std::set<std::string> GetAllFeatureNames() const; 38 std::set<std::string> GetAllFeatureNames() const;
38 39
39 // Gets the feature |feature_name|, if it exists. 40 // Gets the feature |feature_name|, if it exists.
40 virtual Feature* GetFeature(const std::string& feature_name) OVERRIDE; 41 virtual Feature* GetFeature(const std::string& feature_name) OVERRIDE;
41 42
42 private: 43 private:
43 typedef std::map<std::string, linked_ptr<Feature> > FeatureMap; 44 typedef std::map<std::string, linked_ptr<Feature> > FeatureMap;
44 FeatureMap features_; 45 FeatureMap features_;
45 46
46 FeatureFactory factory_; 47 FeatureFactory factory_;
47 }; 48 };
48 49
49 } // namespace extensions 50 } // namespace extensions
50 51
51 #endif // CHROME_COMMON_EXTENSIONS_FEATURES_SIMPLE_FEATURE_PROVIDER_H_ 52 #endif // CHROME_COMMON_EXTENSIONS_FEATURES_BASE_FEATURE_PROVIDER_H_
OLDNEW
« no previous file with comments | « chrome/common/extensions/extension.cc ('k') | chrome/common/extensions/features/base_feature_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698