diff options
Diffstat (limited to 'chrome/browser/extensions/crx_installer.h')
-rw-r--r-- | chrome/browser/extensions/crx_installer.h | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/chrome/browser/extensions/crx_installer.h b/chrome/browser/extensions/crx_installer.h index b1212a2..349b225 100644 --- a/chrome/browser/extensions/crx_installer.h +++ b/chrome/browser/extensions/crx_installer.h @@ -49,11 +49,12 @@ class CrxInstaller // ExtensionFunction to a resulting download in the download manager, it's // currently necessary. This is the |id| of an extension to be installed // *by the web store only* which should not get the permissions install - // prompt. + // prompt. This should only be called on the UI thread. // crbug.com/54916 static void SetWhitelistedInstallId(const std::string& id); - // Returns whether |id| was found and removed (was whitelisted). + // Returns whether |id| was found and removed (was whitelisted). This should + // only be called on the UI thread. static bool ClearWhitelistedInstallId(const std::string& id); // Constructor. Extensions will be unpacked to |install_directory|. @@ -120,13 +121,13 @@ class CrxInstaller // Called after OnUnpackSuccess as a last check to see whether the install // should complete. - bool AllowInstall(Extension* extension, std::string* error); + bool AllowInstall(const Extension* extension, std::string* error); // SandboxedExtensionUnpackerClient virtual void OnUnpackFailure(const std::string& error_message); virtual void OnUnpackSuccess(const FilePath& temp_dir, const FilePath& extension_dir, - Extension* extension); + const Extension* extension); // Runs on the UI thread. Confirms with the user (via ExtensionInstallUI) that // it is OK to install this extension. @@ -187,7 +188,7 @@ class CrxInstaller // The extension we're installing. We own this and either pass it off to // ExtensionsService on success, or delete it on failure. - scoped_ptr<Extension> extension_; + scoped_refptr<const Extension> extension_; // If non-empty, contains the current version of the extension we're // installing (for upgrades). |