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

Side by Side Diff: extensions/common/manifest_handler.h

Issue 51433002: Enable permission warnings from ManifestHandlers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address code review comments. Created 7 years, 1 month 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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 EXTENSIONS_COMMON_MANIFEST_HANDLER_H_ 5 #ifndef EXTENSIONS_COMMON_MANIFEST_HANDLER_H_
6 #define EXTENSIONS_COMMON_MANIFEST_HANDLER_H_ 6 #define EXTENSIONS_COMMON_MANIFEST_HANDLER_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
11 11
12 #include "base/lazy_instance.h" 12 #include "base/lazy_instance.h"
13 #include "base/memory/linked_ptr.h" 13 #include "base/memory/linked_ptr.h"
14 #include "base/strings/string16.h" 14 #include "base/strings/string16.h"
15 #include "extensions/common/manifest.h" 15 #include "extensions/common/manifest.h"
16 16
17 namespace extensions { 17 namespace extensions {
18 class Extension; 18 class Extension;
19 class ManifestPermission;
20 class ManifestPermissionSet;
19 21
20 // An interface for clients that recognize and parse keys in extension 22 // An interface for clients that recognize and parse keys in extension
21 // manifests. 23 // manifests.
22 class ManifestHandler { 24 class ManifestHandler {
23 public: 25 public:
24 ManifestHandler(); 26 ManifestHandler();
25 virtual ~ManifestHandler(); 27 virtual ~ManifestHandler();
26 28
27 // Attempts to parse the extension's manifest. 29 // Attempts to parse the extension's manifest.
28 // Returns true on success or false on failure; if false, |error| will 30 // Returns true on success or false on failure; if false, |error| will
(...skipping 28 matching lines...) Expand all
57 virtual const std::vector<std::string> PrerequisiteKeys() const; 59 virtual const std::vector<std::string> PrerequisiteKeys() const;
58 60
59 // Associate us with our keys() in the manifest. A handler can register 61 // Associate us with our keys() in the manifest. A handler can register
60 // for multiple keys. The global registry takes ownership of this; 62 // for multiple keys. The global registry takes ownership of this;
61 // if it has an existing handler for |key|, it replaces it with this. 63 // if it has an existing handler for |key|, it replaces it with this.
62 // Manifest handlers must be registered at process startup in 64 // Manifest handlers must be registered at process startup in
63 // chrome_manifest_handlers.cc: 65 // chrome_manifest_handlers.cc:
64 // (new MyManifestHandler)->Register(); 66 // (new MyManifestHandler)->Register();
65 void Register(); 67 void Register();
66 68
69 virtual ManifestPermission* CreatePermission();
70 virtual ManifestPermission* CreateInitialRequiredPermission(
71 const Extension* extension);
72
67 // Calling FinalizeRegistration indicates that there are no more 73 // Calling FinalizeRegistration indicates that there are no more
68 // manifest handlers to be registered. 74 // manifest handlers to be registered.
69 static void FinalizeRegistration(); 75 static void FinalizeRegistration();
70 76
71 static bool IsRegistrationFinalized(); 77 static bool IsRegistrationFinalized();
72 78
73 // Call Parse on all registered manifest handlers that should parse 79 // Call Parse on all registered manifest handlers that should parse
74 // this extension. 80 // this extension.
75 static bool ParseExtension(Extension* extension, string16* error); 81 static bool ParseExtension(Extension* extension, string16* error);
76 82
77 // Call Validate on all registered manifest handlers for this extension. 83 // Call Validate on all registered manifest handlers for this extension.
78 static bool ValidateExtension(const Extension* extension, 84 static bool ValidateExtension(const Extension* extension,
79 std::string* error, 85 std::string* error,
80 std::vector<InstallWarning>* warnings); 86 std::vector<InstallWarning>* warnings);
81 87
88 // Creates a |ManifestPermission| instance for the given manifest key |name|.
89 // The returned permission does not contain any permission data, so this
90 // method is usually used before calling |FromValue| or |Read|. Returns
91 // |NULL| if the manifest handler for the given manifest key does not support
92 // custom permissions.
93 static ManifestPermission* CreatePermission(const std::string& name);
94
95 // Adds to |permission_set| the initial set of required manifest permissions
96 // for the given |extension|. Note this should be called after all manifest
97 // data elements have been read, parsed and stored in the manifest data
98 // property of |extension|, as manifest handlers need access to their
99 // manifest data to initialize their required manifest permission.
100 static void AddExtensionRequiredPermissions(
101 const Extension* extension, ManifestPermissionSet* permission_set);
102
82 protected: 103 protected:
83 // A convenience method for handlers that only register for 1 key, 104 // A convenience method for handlers that only register for 1 key,
84 // so that they can define keys() { return SingleKey(kKey); } 105 // so that they can define keys() { return SingleKey(kKey); }
85 static const std::vector<std::string> SingleKey(const std::string& key); 106 static const std::vector<std::string> SingleKey(const std::string& key);
86 107
87 private: 108 private:
88 // The keys to register us for (in Register). 109 // The keys to register us for (in Register).
89 virtual const std::vector<std::string> Keys() const = 0; 110 virtual const std::vector<std::string> Keys() const = 0;
90 }; 111 };
91 112
92 // The global registry for manifest handlers. 113 // The global registry for manifest handlers.
93 class ManifestHandlerRegistry { 114 class ManifestHandlerRegistry {
94 private: 115 private:
95 friend class ManifestHandler; 116 friend class ManifestHandler;
96 friend class ScopedTestingManifestHandlerRegistry; 117 friend class ScopedTestingManifestHandlerRegistry;
97 friend struct base::DefaultLazyInstanceTraits<ManifestHandlerRegistry>; 118 friend struct base::DefaultLazyInstanceTraits<ManifestHandlerRegistry>;
98 119
99 ManifestHandlerRegistry(); 120 ManifestHandlerRegistry();
100 ~ManifestHandlerRegistry(); 121 ~ManifestHandlerRegistry();
101 122
102 void Finalize(); 123 void Finalize();
103 124
104 void RegisterManifestHandler(const std::string& key, 125 void RegisterManifestHandler(const std::string& key,
105 linked_ptr<ManifestHandler> handler); 126 linked_ptr<ManifestHandler> handler);
106 bool ParseExtension(Extension* extension, string16* error); 127 bool ParseExtension(Extension* extension, string16* error);
107 bool ValidateExtension(const Extension* extension, 128 bool ValidateExtension(const Extension* extension,
108 std::string* error, 129 std::string* error,
109 std::vector<InstallWarning>* warnings); 130 std::vector<InstallWarning>* warnings);
110 131
132 ManifestPermission* CreatePermission(const std::string& name);
133
134 void AddExtensionRequiredPermissions(const Extension* extension,
135 ManifestPermissionSet* permission_set);
136
111 // Overrides the current global ManifestHandlerRegistry with 137 // Overrides the current global ManifestHandlerRegistry with
112 // |registry|, returning the current one. 138 // |registry|, returning the current one.
113 static ManifestHandlerRegistry* SetForTesting( 139 static ManifestHandlerRegistry* SetForTesting(
114 ManifestHandlerRegistry* new_registry); 140 ManifestHandlerRegistry* new_registry);
115 141
116 typedef std::map<std::string, linked_ptr<ManifestHandler> > 142 typedef std::map<std::string, linked_ptr<ManifestHandler> >
117 ManifestHandlerMap; 143 ManifestHandlerMap;
118 typedef std::map<ManifestHandler*, int> ManifestHandlerPriorityMap; 144 typedef std::map<ManifestHandler*, int> ManifestHandlerPriorityMap;
119 145
120 // Puts the manifest handlers in order such that each handler comes after 146 // Puts the manifest handlers in order such that each handler comes after
121 // any handlers for their PrerequisiteKeys. If there is no handler for 147 // any handlers for their PrerequisiteKeys. If there is no handler for
122 // a prerequisite key, that dependency is simply ignored. 148 // a prerequisite key, that dependency is simply ignored.
123 // CHECKs that there are no manifest handlers with circular dependencies. 149 // CHECKs that there are no manifest handlers with circular dependencies.
124 void SortManifestHandlers(); 150 void SortManifestHandlers();
125 151
126 // All registered manifest handlers. 152 // All registered manifest handlers.
127 ManifestHandlerMap handlers_; 153 ManifestHandlerMap handlers_;
128 154
129 // The priority for each manifest handler. Handlers with lower priority 155 // The priority for each manifest handler. Handlers with lower priority
130 // values are evaluated first. 156 // values are evaluated first.
131 ManifestHandlerPriorityMap priority_map_; 157 ManifestHandlerPriorityMap priority_map_;
132 158
133 bool is_finalized_; 159 bool is_finalized_;
134 }; 160 };
135 161
136 } // namespace extensions 162 } // namespace extensions
137 163
138 #endif // EXTENSIONS_COMMON_MANIFEST_HANDLER_H_ 164 #endif // EXTENSIONS_COMMON_MANIFEST_HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698