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

Unified Diff: extensions/common/permissions/permission_set.cc

Issue 228343002: Cleanup webRequest/Internal API and permission registration. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: extensions/common/permissions/permission_set.cc
diff --git a/extensions/common/permissions/permission_set.cc b/extensions/common/permissions/permission_set.cc
index fd32d4e83d64adcd6c29405b0f826ef5a6013072..b101e75ff696a860dc4c60b5c5c813f4cbb900b9 100644
--- a/extensions/common/permissions/permission_set.cc
+++ b/extensions/common/permissions/permission_set.cc
@@ -250,15 +250,6 @@ void PermissionSet::InitImplicitPermissions() {
if (apis_.find(APIPermission::kDownloads) != apis_.end())
apis_.insert(APIPermission::kDownloadsInternal);
- // TODO(fsamuel): Is there a better way to request access to the WebRequest
- // API without exposing it to the Chrome App?
- if (apis_.find(APIPermission::kWebView) != apis_.end())
- apis_.insert(APIPermission::kWebRequestInternal);
not at google - send to devlin 2014/04/08 15:09:36 fsamuel: fyi
-
- // The webRequest permission implies the internal version as well.
- if (apis_.find(APIPermission::kWebRequest) != apis_.end())
- apis_.insert(APIPermission::kWebRequestInternal);
-
// The fileBrowserHandler permission implies the internal version as well.
if (apis_.find(APIPermission::kFileBrowserHandler) != apis_.end())
apis_.insert(APIPermission::kFileBrowserHandlerInternal);

Powered by Google App Engine
This is Rietveld 408576698