Cleanup extension permissions module.

chrome/browser/extensions/extension_permissions_api.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_PERMISSIONS_API_H__
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_PERMISSIONS_API_H__
 #pragma once
 
 #include "base/compiler_specific.h"
 #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;
 }
 class Extension;
 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
   // user has already been prompted, if necessary, for the extra permissions.
   void AddPermissions(const Extension* extension,
                       const ExtensionPermissionSet* permissions);
 
   // Removes the set of |permissions| from the |extension|'s active permission
   // set and sends the relevant messages and notifications.
   void RemovePermissions(const Extension* extension,
@@ skipping 58 matching lines @@
   virtual void InstallUIProceed() OVERRIDE;
   virtual void InstallUIAbort(bool user_initiated) OVERRIDE;
 
  protected:
   virtual ~RequestPermissionsFunction();
   virtual bool RunImpl() OVERRIDE;
 
  private:
   scoped_ptr<ExtensionInstallUI> install_ui_;
   scoped_refptr<ExtensionPermissionSet> requested_permissions_;
-  const Extension* extension_;
   DECLARE_EXTENSION_FUNCTION_NAME("permissions.request")
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_PERMISSIONS_API_H__