| Index: chrome/browser/extensions/api/storage/settings_apitest.cc
|
| diff --git a/chrome/browser/extensions/api/storage/settings_apitest.cc b/chrome/browser/extensions/api/storage/settings_apitest.cc
|
| index e909ae08f35e7ebdd02752dac9347a0e1a1a7160..d0fb0f1a902465b96f94ae8fc603aa87d08799d7 100644
|
| --- a/chrome/browser/extensions/api/storage/settings_apitest.cc
|
| +++ b/chrome/browser/extensions/api/storage/settings_apitest.cc
|
| @@ -12,8 +12,19 @@
|
| #include "chrome/browser/extensions/api/storage/settings_sync_util.h"
|
| #include "chrome/browser/extensions/extension_apitest.h"
|
| #include "chrome/browser/extensions/extension_system_factory.h"
|
| +#include "chrome/browser/policy/schema_registry_service.h"
|
| +#include "chrome/browser/policy/schema_registry_service_factory.h"
|
| #include "chrome/browser/profiles/profile.h"
|
| #include "chrome/browser/ui/browser.h"
|
| +#include "components/policy/core/browser/browser_policy_connector.h"
|
| +#include "components/policy/core/common/mock_configuration_policy_provider.h"
|
| +#include "components/policy/core/common/policy_bundle.h"
|
| +#include "components/policy/core/common/policy_map.h"
|
| +#include "components/policy/core/common/policy_namespace.h"
|
| +#include "components/policy/core/common/policy_types.h"
|
| +#include "components/policy/core/common/schema.h"
|
| +#include "components/policy/core/common/schema_map.h"
|
| +#include "components/policy/core/common/schema_registry.h"
|
| #include "extensions/browser/api/storage/settings_namespace.h"
|
| #include "extensions/browser/api/storage/storage_frontend.h"
|
| #include "extensions/browser/extension_system.h"
|
| @@ -29,20 +40,6 @@
|
| #include "sync/api/syncable_service.h"
|
| #include "testing/gmock/include/gmock/gmock.h"
|
|
|
| -#if defined(ENABLE_CONFIGURATION_POLICY)
|
| -#include "chrome/browser/policy/schema_registry_service.h"
|
| -#include "chrome/browser/policy/schema_registry_service_factory.h"
|
| -#include "components/policy/core/browser/browser_policy_connector.h"
|
| -#include "components/policy/core/common/mock_configuration_policy_provider.h"
|
| -#include "components/policy/core/common/policy_bundle.h"
|
| -#include "components/policy/core/common/policy_map.h"
|
| -#include "components/policy/core/common/policy_namespace.h"
|
| -#include "components/policy/core/common/policy_types.h"
|
| -#include "components/policy/core/common/schema.h"
|
| -#include "components/policy/core/common/schema_map.h"
|
| -#include "components/policy/core/common/schema_registry.h"
|
| -#endif
|
| -
|
| namespace extensions {
|
|
|
| using settings_namespace::LOCAL;
|
| @@ -79,13 +76,11 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
|
| void SetUpInProcessBrowserTestFixture() override {
|
| ExtensionApiTest::SetUpInProcessBrowserTestFixture();
|
|
|
| -#if defined(ENABLE_CONFIGURATION_POLICY)
|
| EXPECT_CALL(policy_provider_, IsInitializationComplete(_))
|
| .WillRepeatedly(Return(true));
|
| policy_provider_.SetAutoRefresh();
|
| policy::BrowserPolicyConnector::SetPolicyProviderForTesting(
|
| &policy_provider_);
|
| -#endif
|
| }
|
|
|
| void ReplyWhenSatisfied(
|
| @@ -132,7 +127,6 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
|
| SendChangesToSyncableService(change_list, GetSyncableService());
|
| }
|
|
|
| -#if defined(ENABLE_CONFIGURATION_POLICY)
|
| void SetPolicies(const base::DictionaryValue& policies) {
|
| scoped_ptr<policy::PolicyBundle> bundle(new policy::PolicyBundle());
|
| policy::PolicyMap& policy_map = bundle->Get(policy::PolicyNamespace(
|
| @@ -141,7 +135,6 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
|
| policy::POLICY_SCOPE_USER, policy::POLICY_SOURCE_CLOUD);
|
| policy_provider_.UpdatePolicy(std::move(bundle));
|
| }
|
| -#endif
|
|
|
| private:
|
| const Extension* MaybeLoadAndReplyWhenSatisfied(
|
| @@ -210,9 +203,7 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
|
| }
|
|
|
| protected:
|
| -#if defined(ENABLE_CONFIGURATION_POLICY)
|
| policy::MockConfigurationPolicyProvider policy_provider_;
|
| -#endif
|
| };
|
|
|
| IN_PROC_BROWSER_TEST_F(ExtensionSettingsApiTest, SimpleTest) {
|
| @@ -409,15 +400,9 @@ IN_PROC_BROWSER_TEST_F(ExtensionSettingsApiTest, IsStorageEnabled) {
|
| EXPECT_TRUE(frontend->IsStorageEnabled(LOCAL));
|
| EXPECT_TRUE(frontend->IsStorageEnabled(SYNC));
|
|
|
| -#if defined(ENABLE_CONFIGURATION_POLICY)
|
| EXPECT_TRUE(frontend->IsStorageEnabled(MANAGED));
|
| -#else
|
| - EXPECT_FALSE(frontend->IsStorageEnabled(MANAGED));
|
| -#endif
|
| }
|
|
|
| -#if defined(ENABLE_CONFIGURATION_POLICY)
|
| -
|
| IN_PROC_BROWSER_TEST_F(ExtensionSettingsApiTest, ExtensionsSchemas) {
|
| // Verifies that the Schemas for the extensions domain are created on startup.
|
| Profile* profile = browser()->profile();
|
| @@ -575,11 +560,8 @@ IN_PROC_BROWSER_TEST_F(ExtensionSettingsApiTest,
|
| EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
|
| }
|
|
|
| -#endif // defined(ENABLE_CONFIGURATION_POLICY)
|
| -
|
| IN_PROC_BROWSER_TEST_F(ExtensionSettingsApiTest, ManagedStorageDisabled) {
|
| - // Disable the 'managed' namespace. This is redundant when
|
| - // ENABLE_CONFIGURATION_POLICY is not defined.
|
| + // Disable the 'managed' namespace.
|
| StorageFrontend* frontend = StorageFrontend::Get(browser()->profile());
|
| frontend->DisableStorageForTesting(MANAGED);
|
| EXPECT_FALSE(frontend->IsStorageEnabled(MANAGED));
|
|
|