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

Unified Diff: chrome/browser/extensions/extension_service.h

Issue 8493017: Cleanup extension permissions module. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_service.h
diff --git a/chrome/browser/extensions/extension_service.h b/chrome/browser/extensions/extension_service.h
index f5bda4f3364cee6434af880bc84e1afd03d2a968..6d7923a96f19d491971db2e3eee336b21a55f8cd 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -25,7 +25,6 @@
#include "chrome/browser/extensions/apps_promo.h"
#include "chrome/browser/extensions/extension_icon_manager.h"
#include "chrome/browser/extensions/extension_menu_manager.h"
-#include "chrome/browser/extensions/extension_permissions_api.h"
#include "chrome/browser/extensions/extension_prefs.h"
#include "chrome/browser/extensions/extension_process_manager.h"
#include "chrome/browser/extensions/extension_sync_data.h"
@@ -324,18 +323,10 @@ class ExtensionService
virtual void DisableExtension(const std::string& extension_id);
// Updates the |extension|'s granted permissions lists to include all
- // permissions in the |extension|'s manifest.
- void GrantPermissions(const Extension* extension);
-
- // Updates the |extension|'s granted permissions lists to include all
// permissions in the |extension|'s manifest and re-enables the
// extension.
void GrantPermissionsAndEnableExtension(const Extension* extension);
- // Sets the |extension|'s active permissions to |permissions|.
- void UpdateActivePermissions(const Extension* extension,
- const ExtensionPermissionSet* permissions);
-
// Check for updates (or potentially new extensions from external providers)
void CheckForExternalUpdates();
@@ -463,10 +454,6 @@ class ExtensionService
return app_notification_manager_.get();
}
- ExtensionPermissionsManager* permissions_manager() {
- return &permissions_manager_;
- }
-
ExtensionBrowserEventRouter* browser_event_router() {
return browser_event_router_.get();
}
@@ -784,9 +771,6 @@ class ExtensionService
// Keeps track of app notifications.
scoped_refptr<AppNotificationManager> app_notification_manager_;
- // Keeps track of extension permissions.
- ExtensionPermissionsManager permissions_manager_;
-
// Keeps track of favicon-sized omnibox icons for extensions.
ExtensionIconManager omnibox_icon_manager_;
ExtensionIconManager omnibox_popup_icon_manager_;

Powered by Google App Engine
This is Rietveld 408576698