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

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

Issue 3522015: Implement new strategy for default apps (Closed)
Patch Set: all done Created 10 years, 2 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
« no previous file with comments | « chrome/browser/extensions/extension_updater.cc ('k') | chrome/browser/extensions/extensions_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extension_updater_unittest.cc
diff --git a/chrome/browser/extensions/extension_updater_unittest.cc b/chrome/browser/extensions/extension_updater_unittest.cc
index b40eb6ecc2f206ede1668949a456105fad273307..2636ccbaf2f24e7d63305d5a42718b20ffce9639 100644
--- a/chrome/browser/extensions/extension_updater_unittest.cc
+++ b/chrome/browser/extensions/extension_updater_unittest.cc
@@ -130,7 +130,7 @@ void CreateTestPendingExtensions(int count, const GURL& update_url,
(*pending_extensions)[id] =
PendingExtensionInfo(update_url, crx_type, kIsFromSync,
kInstallSilently, kInitialState,
- kInitialIncognitoEnabled);
+ kInitialIncognitoEnabled, Extension::INTERNAL);
}
}
@@ -568,7 +568,7 @@ class ExtensionUpdaterTest : public testing::Test {
pending_extensions[id] =
PendingExtensionInfo(test_url, kExpectedCrxType, kIsFromSync,
kInstallSilently, kInitialState,
- kInitialIncognitoEnabled);
+ kInitialIncognitoEnabled, Extension::INTERNAL);
service.set_pending_extensions(pending_extensions);
}
@@ -897,13 +897,15 @@ TEST(ExtensionUpdaterTest, TestManifestFetchesBuilderAddExtension) {
builder.AddPendingExtension(
GenerateId("foo"), PendingExtensionInfo(GURL("http:google.com:foo"),
PendingExtensionInfo::EXTENSION,
- false, false, true, false));
+ false, false, true, false,
+ Extension::INTERNAL));
EXPECT_TRUE(builder.GetFetches().empty());
// Extensions with empty IDs should be rejected.
builder.AddPendingExtension(
"", PendingExtensionInfo(GURL(), PendingExtensionInfo::EXTENSION,
- false, false, true, false));
+ false, false, true, false,
+ Extension::INTERNAL));
EXPECT_TRUE(builder.GetFetches().empty());
// TODO(akalin): Test that extensions with empty update URLs
@@ -914,7 +916,8 @@ TEST(ExtensionUpdaterTest, TestManifestFetchesBuilderAddExtension) {
builder.AddPendingExtension(
GenerateId("foo"), PendingExtensionInfo(GURL(),
PendingExtensionInfo::EXTENSION,
- false, false, true, false));
+ false, false, true, false,
+ Extension::INTERNAL));
std::vector<ManifestFetchData*> fetches = builder.GetFetches();
ASSERT_EQ(1u, fetches.size());
scoped_ptr<ManifestFetchData> fetch(fetches[0]);
« no previous file with comments | « chrome/browser/extensions/extension_updater.cc ('k') | chrome/browser/extensions/extensions_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698