Cleanup extension permissions module.

chrome/browser/extensions/extension_permissions_api.h

Index: chrome/browser/extensions/extension_permissions_api.h
diff --git a/chrome/browser/extensions/extension_permissions_api.h b/chrome/browser/extensions/extension_permissions_api.h
index 7644a6a7b824ce301616800de095a1d9d2df0253..319d0252924de0cee6ee3b723e69f798b190c255 100644
--- a/chrome/browser/extensions/extension_permissions_api.h
+++ b/chrome/browser/extensions/extension_permissions_api.h
@@ -10,8 +10,6 @@
 #include "chrome/browser/extensions/extension_function.h"
 #include "chrome/browser/extensions/extension_install_ui.h"
 #include "chrome/common/extensions/extension_permission_set.h"
-#include "chrome/common/chrome_notification_types.h"
-#include "content/browser/renderer_host/render_process_host.h"
 
 namespace base {
 class DictionaryValue;
@@ -21,10 +19,10 @@ class ExtensionPermissionSet;
 class ExtensionService;
 class Profile;
 
-class ExtensionPermissionsManager {
+class ExtensionPermissionsUpdater {

[Aaron Boodman, 2011/11/14 19:26:05]

In general, it's not great to have more than one top-level class in a .h file
because it makes it harder to find definitions when reading sources.

Can you move this out into its own file?

[jstritar, 2011/12/12 22:28:03]

Done.

  public:
-  explicit ExtensionPermissionsManager(ExtensionService* extension_service);
-  ~ExtensionPermissionsManager();
+  explicit ExtensionPermissionsUpdater(ExtensionService* extension_service);
+  ~ExtensionPermissionsUpdater();
 
   // Adds the set of |permissions| to the |extension|'s active permission set
   // and sends the relevant messages and notifications. This method assumes the
@@ -103,7 +101,6 @@ class RequestPermissionsFunction : public AsyncExtensionFunction,
  private:
   scoped_ptr<ExtensionInstallUI> install_ui_;
   scoped_refptr<ExtensionPermissionSet> requested_permissions_;
-  const Extension* extension_;
   DECLARE_EXTENSION_FUNCTION_NAME("permissions.request")
 };