| Index: chrome/browser/extensions/extension_webstore_private_api.cc
|
| diff --git a/chrome/browser/extensions/extension_webstore_private_api.cc b/chrome/browser/extensions/extension_webstore_private_api.cc
|
| index 19c4597de8877cd83c5a4d5aa4b4abd3bf9ecc3c..8fab25b46190ecb9c03787d54b3de957a8a83af6 100644
|
| --- a/chrome/browser/extensions/extension_webstore_private_api.cc
|
| +++ b/chrome/browser/extensions/extension_webstore_private_api.cc
|
| @@ -57,15 +57,6 @@ ProfileSyncService* test_sync_service = NULL;
|
| BrowserSignin* test_signin = NULL;
|
| bool ignore_user_gesture_for_tests = false;
|
|
|
| -// A flag used for BeginInstallWithManifest::SetAutoConfirmForTests.
|
| -enum AutoConfirmForTest {
|
| - DO_NOT_SKIP = 0,
|
| - PROCEED,
|
| - ABORT
|
| -};
|
| -AutoConfirmForTest auto_confirm_for_tests = DO_NOT_SKIP;
|
| -
|
| -
|
| // Returns either the test sync service, or the real one from |profile|.
|
| ProfileSyncService* GetSyncService(Profile* profile) {
|
| if (test_sync_service)
|
| @@ -270,62 +261,27 @@ void BeginInstallWithManifestFunction::SetIgnoreUserGestureForTests(
|
| ignore_user_gesture_for_tests = ignore;
|
| }
|
|
|
| -void BeginInstallWithManifestFunction::SetAutoConfirmForTests(
|
| - bool should_proceed) {
|
| - auto_confirm_for_tests = should_proceed ? PROCEED : ABORT;
|
| -}
|
| -
|
| void BeginInstallWithManifestFunction::OnWebstoreParseSuccess(
|
| const SkBitmap& icon, DictionaryValue* parsed_manifest) {
|
| CHECK(parsed_manifest);
|
| icon_ = icon;
|
| parsed_manifest_.reset(parsed_manifest);
|
|
|
| - // If we were passed a localized name to use in the dialog, create a copy
|
| - // of the original manifest and replace the name in it.
|
| - scoped_ptr<DictionaryValue> localized_manifest;
|
| - if (!localized_name_.empty()) {
|
| - localized_manifest.reset(parsed_manifest->DeepCopy());
|
| - localized_manifest->SetString(extension_manifest_keys::kName,
|
| - localized_name_);
|
| - }
|
| -
|
| - // Create a dummy extension and show the extension install confirmation
|
| - // dialog.
|
| - std::string init_errors;
|
| - dummy_extension_ = Extension::CreateWithId(
|
| - FilePath(),
|
| - Extension::INTERNAL,
|
| - localized_manifest.get() ? *localized_manifest.get() : *parsed_manifest,
|
| - Extension::NO_FLAGS,
|
| + ShowExtensionInstallDialogForManifest(
|
| + profile(),
|
| + this,
|
| + parsed_manifest,
|
| id_,
|
| - &init_errors);
|
| + localized_name_,
|
| + &icon_,
|
| + ExtensionInstallUI::INSTALL_PROMPT,
|
| + &dummy_extension_);
|
| if (!dummy_extension_.get()) {
|
| OnWebstoreParseFailure(WebstoreInstallHelper::Delegate::MANIFEST_ERROR,
|
| kInvalidManifestError);
|
| return;
|
| }
|
|
|
| - if (icon_.empty())
|
| - icon_ = Extension::GetDefaultIcon(dummy_extension_->is_app());
|
| -
|
| - // In tests, we may have setup to proceed or abort without putting up the real
|
| - // confirmation dialog.
|
| - if (auto_confirm_for_tests != DO_NOT_SKIP) {
|
| - if (auto_confirm_for_tests == PROCEED)
|
| - this->InstallUIProceed();
|
| - else
|
| - this->InstallUIAbort(true);
|
| - return;
|
| - }
|
| -
|
| - ShowExtensionInstallDialog(profile(),
|
| - this,
|
| - dummy_extension_.get(),
|
| - &icon_,
|
| - dummy_extension_->GetPermissionMessageStrings(),
|
| - ExtensionInstallUI::INSTALL_PROMPT);
|
| -
|
| // Control flow finishes up in InstallUIProceed or InstallUIAbort.
|
| }
|
|
|
|
|