Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(675)

Unified Diff: chrome/browser/extensions/updater/extension_updater_unittest.cc

Issue 2299203004: Attempt to repair corrupt enterprise policy force-installed extensions (Closed)
Patch Set: switched to using installsource, addressed review comments Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/updater/extension_updater_unittest.cc
diff --git a/chrome/browser/extensions/updater/extension_updater_unittest.cc b/chrome/browser/extensions/updater/extension_updater_unittest.cc
index 33ae029f269379838d8a8a9708bbc08aa92ef1f5..0ae32f8ab65adfec645a719606cb7d09e1b759a8 100644
--- a/chrome/browser/extensions/updater/extension_updater_unittest.cc
+++ b/chrome/browser/extensions/updater/extension_updater_unittest.cc
@@ -2131,22 +2131,22 @@ TEST_F(ExtensionUpdaterTest, TestManifestFetchesBuilderAddExtension) {
// the delegate.
std::string id = crx_file::id_util::GenerateId("foo");
EXPECT_CALL(delegate, GetPingDataForExtension(id, _)).WillOnce(Return(false));
- EXPECT_TRUE(
- downloader->AddPendingExtension(id, GURL("http://example.com/update"),
- 0));
+ EXPECT_TRUE(downloader->AddPendingExtension(
+ id, GURL("http://example.com/update"), false, 0));
downloader->StartAllPending(NULL);
Mock::VerifyAndClearExpectations(&delegate);
EXPECT_EQ(1u, ManifestFetchersCount(downloader.get()));
// Extensions with invalid update URLs should be rejected.
id = crx_file::id_util::GenerateId("foo2");
- EXPECT_FALSE(
- downloader->AddPendingExtension(id, GURL("http:google.com:foo"), 0));
+ EXPECT_FALSE(downloader->AddPendingExtension(id, GURL("http:google.com:foo"),
+ false, 0));
downloader->StartAllPending(NULL);
EXPECT_EQ(1u, ManifestFetchersCount(downloader.get()));
// Extensions with empty IDs should be rejected.
- EXPECT_FALSE(downloader->AddPendingExtension(std::string(), GURL(), 0));
+ EXPECT_FALSE(
+ downloader->AddPendingExtension(std::string(), GURL(), false, 0));
downloader->StartAllPending(NULL);
EXPECT_EQ(1u, ManifestFetchersCount(downloader.get()));
@@ -2162,7 +2162,7 @@ TEST_F(ExtensionUpdaterTest, TestManifestFetchesBuilderAddExtension) {
// filled in.
id = crx_file::id_util::GenerateId("foo3");
EXPECT_CALL(delegate, GetPingDataForExtension(id, _)).WillOnce(Return(false));
- EXPECT_TRUE(downloader->AddPendingExtension(id, GURL(), 0));
+ EXPECT_TRUE(downloader->AddPendingExtension(id, GURL(), false, 0));
downloader->StartAllPending(NULL);
EXPECT_EQ(1u, ManifestFetchersCount(downloader.get()));
« no previous file with comments | « chrome/browser/extensions/updater/extension_updater.cc ('k') | extensions/browser/updater/extension_downloader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698