Index: chrome/browser/extensions/api/webstore_private/webstore_private_apitest.cc |
diff --git a/chrome/browser/extensions/api/webstore_private/webstore_private_apitest.cc b/chrome/browser/extensions/api/webstore_private/webstore_private_apitest.cc |
index bb2d0690f5924070dba03a420a2ef1b13bf9c106..174024df5501bfe0cfb08f8a97616599ecd48599 100644 |
--- a/chrome/browser/extensions/api/webstore_private/webstore_private_apitest.cc |
+++ b/chrome/browser/extensions/api/webstore_private/webstore_private_apitest.cc |
@@ -292,21 +292,11 @@ IN_PROC_BROWSER_TEST_F(ExtensionWebstorePrivateApiTest, InstallCancelled) { |
} |
IN_PROC_BROWSER_TEST_F(ExtensionWebstorePrivateApiTest, IncorrectManifest1) { |
- WebstoreInstallListener listener; |
- WebstorePrivateApi::SetWebstoreInstallerDelegateForTesting(&listener); |
ASSERT_TRUE(RunInstallTest("incorrect_manifest1.html", "extension.crx")); |
- listener.Wait(); |
- ASSERT_TRUE(listener.received_failure()); |
- ASSERT_EQ("Manifest file is invalid.", listener.error()); |
} |
IN_PROC_BROWSER_TEST_F(ExtensionWebstorePrivateApiTest, IncorrectManifest2) { |
- WebstoreInstallListener listener; |
- WebstorePrivateApi::SetWebstoreInstallerDelegateForTesting(&listener); |
ASSERT_TRUE(RunInstallTest("incorrect_manifest2.html", "extension.crx")); |
- listener.Wait(); |
- EXPECT_TRUE(listener.received_failure()); |
- ASSERT_EQ("Manifest file is invalid.", listener.error()); |
} |
// Tests that we can request an app installed bubble (instead of the default |
@@ -356,6 +346,11 @@ IN_PROC_BROWSER_TEST_F(ExtensionWebstorePrivateApiTest, InstallTheme) { |
ASSERT_EQ("iamefpfkojoapidjnbafmgkgncegbkad", listener.id()); |
} |
+// Tests that an error is properly reported when an empty crx is returned. |
+IN_PROC_BROWSER_TEST_F(ExtensionWebstorePrivateApiTest, EmptyCrx) { |
+ ASSERT_TRUE(RunInstallTest("empty.html", "empty.crx")); |
+} |
+ |
// Tests successfully installing a bundle of 2 apps and 2 extensions. |
IN_PROC_BROWSER_TEST_F(ExtensionWebstorePrivateBundleTest, InstallBundle) { |
extensions::BundleInstaller::SetAutoApproveForTesting(true); |