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

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: merge Created 8 years, 11 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: chrome/browser/extensions/extension_service.h
diff --git a/chrome/browser/extensions/extension_service.h b/chrome/browser/extensions/extension_service.h
index 3ce1a994e36a8217cbcb28eb4473eb07d116fa22..99229766899155336f9afa7e212f900093b5b29a 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -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"
@@ -349,18 +348,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();
@@ -476,10 +467,6 @@ class ExtensionService
return app_notification_manager_.get();
}
- ExtensionPermissionsManager* permissions_manager() {
- return &permissions_manager_;
- }
-
ExtensionBrowserEventRouter* browser_event_router() {
return browser_event_router_.get();
}
@@ -796,9 +783,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_;
« no previous file with comments | « chrome/browser/extensions/extension_process_manager.cc ('k') | chrome/browser/extensions/extension_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698