Chromium Code Reviews| Index: chrome/browser/extensions/webstore_standalone_installer.cc |
| diff --git a/chrome/browser/extensions/webstore_standalone_installer.cc b/chrome/browser/extensions/webstore_standalone_installer.cc |
| index dc9ea0efdf251a5aa54c0c03d6b66805291d7973..dd2ac5ccfd233b630ba2963374373d272541be59 100644 |
| --- a/chrome/browser/extensions/webstore_standalone_installer.cc |
| +++ b/chrome/browser/extensions/webstore_standalone_installer.cc |
| @@ -132,7 +132,7 @@ void WebstoreStandaloneInstaller::CompleteInstall( |
| void WebstoreStandaloneInstaller::ProceedWithInstallPrompt() { |
| install_prompt_ = CreateInstallPrompt(); |
| - if (install_prompt_) { |
| + if (install_prompt_.get()) { |
| ShowInstallUI(); |
| // Control flow finishes up in InstallUIProceed or InstallUIAbort. |
| } else { |
| @@ -349,7 +349,8 @@ void WebstoreStandaloneInstaller::InstallUIProceed() { |
| // If the target extension has already been installed ephemerally and is |
| // up to date, it can be promoted to a regular installed extension and |
| // downloading from the Web Store is not necessary. |
| - const Extension* extension_to_install = GetLocalizedExtensionForDisplay(); |
| + scoped_refptr<const Extension> extension_to_install = |
|
dcheng
2014/08/26 01:30:17
Note: I missed this in my original overview of the
|
| + GetLocalizedExtensionForDisplay(); |
| if (!extension_to_install) { |
| CompleteInstall(webstore_install::INVALID_MANIFEST, |
| kInvalidManifestError); |
| @@ -421,7 +422,8 @@ void WebstoreStandaloneInstaller::OnExtensionInstallFailure( |
| } |
| void WebstoreStandaloneInstaller::ShowInstallUI() { |
| - const Extension* localized_extension = GetLocalizedExtensionForDisplay(); |
| + scoped_refptr<const Extension> localized_extension = |
| + GetLocalizedExtensionForDisplay(); |
| if (!localized_extension) { |
| CompleteInstall(webstore_install::INVALID_MANIFEST, kInvalidManifestError); |
| return; |