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

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

Issue 2054773002: Replace the WAS_INSTALLED_BY_CUSTODIAN creation flag with a pref (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Response to code review by treib@ Created 4 years, 6 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/extension_service_unittest.cc
diff --git a/chrome/browser/extensions/extension_service_unittest.cc b/chrome/browser/extensions/extension_service_unittest.cc
index 695a39769bda59edd873808396c7e6c352f761fe..f6071d3887501da7d04ee281836fd0d0d20349b2 100644
--- a/chrome/browser/extensions/extension_service_unittest.cc
+++ b/chrome/browser/extensions/extension_service_unittest.cc
@@ -2485,7 +2485,6 @@ TEST_F(ExtensionServiceTest, AddPendingExtensionFromSync) {
const std::string kFakeId(all_zero);
const GURL kFakeUpdateURL("http:://fake.update/url");
const bool kFakeRemoteInstall(false);
- const bool kFakeInstalledByCustodian(false);
EXPECT_TRUE(
service()->pending_extension_manager()->AddFromSync(
@@ -2493,8 +2492,7 @@ TEST_F(ExtensionServiceTest, AddPendingExtensionFromSync) {
kFakeUpdateURL,
base::Version(),
&IsExtension,
- kFakeRemoteInstall,
- kFakeInstalledByCustodian));
+ kFakeRemoteInstall));
const extensions::PendingExtensionInfo* pending_extension_info;
ASSERT_TRUE((pending_extension_info =
@@ -2519,7 +2517,6 @@ const char kGoodUpdateURL[] = "http://good.update/url";
const char kGoodVersion[] = "1";
const bool kGoodIsFromSync = true;
const bool kGoodRemoteInstall = false;
-const bool kGoodInstalledByCustodian = false;
} // namespace
// Test installing a pending extension (this goes through
@@ -2532,8 +2529,7 @@ TEST_F(ExtensionServiceTest, UpdatePendingExtension) {
GURL(kGoodUpdateURL),
base::Version(kGoodVersion),
&IsExtension,
- kGoodRemoteInstall,
- kGoodInstalledByCustodian));
+ kGoodRemoteInstall));
EXPECT_TRUE(service()->pending_extension_manager()->IsIdPending(kGoodId));
base::FilePath path = data_dir().AppendASCII("good.crx");
@@ -2556,8 +2552,7 @@ TEST_F(ExtensionServiceTest, UpdatePendingExtensionWrongVersion) {
GURL(kGoodUpdateURL),
other_version,
&IsExtension,
- kGoodRemoteInstall,
- kGoodInstalledByCustodian));
+ kGoodRemoteInstall));
EXPECT_TRUE(service()->pending_extension_manager()->IsIdPending(kGoodId));
base::FilePath path = data_dir().AppendASCII("good.crx");
@@ -2588,7 +2583,7 @@ bool IsTheme(const Extension* extension) {
TEST_F(ExtensionServiceTest, DISABLED_UpdatePendingTheme) {
InitializeEmptyExtensionService();
EXPECT_TRUE(service()->pending_extension_manager()->AddFromSync(
- theme_crx, GURL(), base::Version(), &IsTheme, false, false));
+ theme_crx, GURL(), base::Version(), &IsTheme, false));
EXPECT_TRUE(service()->pending_extension_manager()->IsIdPending(theme_crx));
base::FilePath path = data_dir().AppendASCII("theme.crx");
@@ -2653,8 +2648,7 @@ TEST_F(ExtensionServiceTest, UpdatePendingExternalCrxWinsOverSync) {
GURL(kGoodUpdateURL),
base::Version(),
&IsExtension,
- kGoodRemoteInstall,
- kGoodInstalledByCustodian));
+ kGoodRemoteInstall));
// Check that there is a pending crx, with is_from_sync set to true.
const extensions::PendingExtensionInfo* pending_extension_info;
@@ -2685,8 +2679,7 @@ TEST_F(ExtensionServiceTest, UpdatePendingExternalCrxWinsOverSync) {
GURL(kGoodUpdateURL),
base::Version(),
&IsExtension,
- kGoodRemoteInstall,
- kGoodInstalledByCustodian));
+ kGoodRemoteInstall));
// Check that the external, non-sync update was not overridden.
ASSERT_TRUE((pending_extension_info =
@@ -2701,7 +2694,7 @@ TEST_F(ExtensionServiceTest, UpdatePendingExternalCrxWinsOverSync) {
TEST_F(ExtensionServiceTest, UpdatePendingCrxThemeMismatch) {
InitializeEmptyExtensionService();
EXPECT_TRUE(service()->pending_extension_manager()->AddFromSync(
- theme_crx, GURL(), base::Version(), &IsExtension, false, false));
+ theme_crx, GURL(), base::Version(), &IsExtension, false));
EXPECT_TRUE(service()->pending_extension_manager()->IsIdPending(theme_crx));
@@ -2728,8 +2721,7 @@ TEST_F(ExtensionServiceTest, UpdatePendingExtensionFailedShouldInstallTest) {
GURL(kGoodUpdateURL),
base::Version(),
&IsTheme,
- kGoodRemoteInstall,
- kGoodInstalledByCustodian));
+ kGoodRemoteInstall));
EXPECT_TRUE(service()->pending_extension_manager()->IsIdPending(kGoodId));
base::FilePath path = data_dir().AppendASCII("good.crx");
@@ -5769,8 +5761,7 @@ class ExtensionSourcePriorityTest : public ExtensionServiceTest {
GURL(kGoodUpdateURL),
base::Version(),
&IsExtension,
- kGoodRemoteInstall,
- kGoodInstalledByCustodian);
+ kGoodRemoteInstall);
}
// Fake a policy install.

Powered by Google App Engine
This is Rietveld 408576698