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

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

Issue 14813039: Retire the Enterprise Web Store policies. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Created 7 years, 7 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/component_loader_unittest.cc
diff --git a/chrome/browser/extensions/component_loader_unittest.cc b/chrome/browser/extensions/component_loader_unittest.cc
index ef61a0b973bc01dbda4c80744423c5bc1b6606a7..ab5da6171b2a01e0029565b84fd724f6676b5164 100644
--- a/chrome/browser/extensions/component_loader_unittest.cc
+++ b/chrome/browser/extensions/component_loader_unittest.cc
@@ -92,16 +92,6 @@ class ComponentLoaderTest : public testing::Test {
extension_path_.Append(kManifestFilename),
&manifest_contents_));
- // Register the user prefs that ComponentLoader will read.
- prefs_.registry()->RegisterStringPref(
- prefs::kEnterpriseWebStoreURL,
- std::string(),
- user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
- prefs_.registry()->RegisterStringPref(
- prefs::kEnterpriseWebStoreName,
- std::string(),
- user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
-
// Register the local state prefs.
#if defined(OS_CHROMEOS)
local_state_.registry()->RegisterBooleanPref(
@@ -260,31 +250,6 @@ TEST_F(ComponentLoaderTest, RemoveAll) {
EXPECT_EQ(0u, extension_service_.extensions()->size());
}
-TEST_F(ComponentLoaderTest, EnterpriseWebStore) {
- component_loader_.AddDefaultComponentExtensions(false);
- component_loader_.LoadAll();
- unsigned int default_count = extension_service_.extensions()->size();
-
- // Set the pref, and it should get loaded automatically.
- extension_service_.set_ready(true);
- prefs_.SetUserPref(prefs::kEnterpriseWebStoreURL,
- Value::CreateStringValue("http://www.google.com"));
- EXPECT_EQ(default_count + 1, extension_service_.extensions()->size());
-
- // Now that the pref is set, check if it's added by default.
- extension_service_.set_ready(false);
- extension_service_.clear_extensions();
- component_loader_.ClearAllRegistered();
- component_loader_.AddDefaultComponentExtensions(false);
- component_loader_.LoadAll();
- EXPECT_EQ(default_count + 1, extension_service_.extensions()->size());
-
- // Number of loaded extensions should be the same after changing the pref.
- prefs_.SetUserPref(prefs::kEnterpriseWebStoreURL,
- Value::CreateStringValue("http://www.google.de"));
- EXPECT_EQ(default_count + 1, extension_service_.extensions()->size());
-}
-
TEST_F(ComponentLoaderTest, AddOrReplace) {
EXPECT_EQ(0u, component_loader_.registered_extensions_count());
component_loader_.AddDefaultComponentExtensions(false);
« no previous file with comments | « chrome/browser/extensions/component_loader.cc ('k') | chrome/browser/policy/configuration_policy_handler_list.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698