diff options
author | mihaip@chromium.org <mihaip@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-17 20:25:49 +0000 |
---|---|---|
committer | mihaip@chromium.org <mihaip@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-17 20:25:49 +0000 |
commit | 6824c9f3070117dd340b60add331155ecb487e02 (patch) | |
tree | 7b6e13732a7c6c09b9fc3bd1d2c0e21527ff25d6 /chrome/browser/extensions/crx_installer_browsertest.cc | |
parent | 5edc18ad4b7b374500d3e9f3df19c3643a23dc9d (diff) | |
download | chromium_src-6824c9f3070117dd340b60add331155ecb487e02.zip chromium_src-6824c9f3070117dd340b60add331155ecb487e02.tar.gz chromium_src-6824c9f3070117dd340b60add331155ecb487e02.tar.bz2 |
Add webstorePrivate.beginInstallWithManifest3 and remove webstorePrivate.beginInstall.
beginInstallWithManifest3 is a superset of beginInstallWithManifest2 with the
following changes:
- Don't require a user gesture (there's a prompt already, and the gesture
requirement is preventing the webstore from lazily loading manifest data).
- Allow additional properties in the object parameter that's passed in. We
had issues where we added the iconUrl param in M15, but the store couldn't
pass it to M14 builds since they barfed on unknown properties.
Also removes the old beginInstall method, since the store always uses
beginInstallWithManifest2 now.
BUG=100242,100240
R=mpcomplete@chromium.org
Review URL: http://codereview.chromium.org/8271020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105899 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/crx_installer_browsertest.cc')
-rw-r--r-- | chrome/browser/extensions/crx_installer_browsertest.cc | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/chrome/browser/extensions/crx_installer_browsertest.cc b/chrome/browser/extensions/crx_installer_browsertest.cc index 96052f1..9353aa1 100644 --- a/chrome/browser/extensions/crx_installer_browsertest.cc +++ b/chrome/browser/extensions/crx_installer_browsertest.cc @@ -67,10 +67,6 @@ class ExtensionCrxInstallerTest : public ExtensionBrowserTest { }; IN_PROC_BROWSER_TEST_F(ExtensionCrxInstallerTest, Whitelisting) { - // We're deprecating this whitelist mechanism, but right now we just assert - // that it actually did prompt. - EXPECT_TRUE(DidWhitelistInstallPrompt("good.crx", - "ldnnhddmnhbkjipkidpdiheffobcpfmf")); #if !defined(OS_CHROMEOS) // An extension with NPAPI should give a prompt. EXPECT_TRUE(DidWhitelistInstallPrompt("uitest/plugins.crx", |