| Index: chrome/browser/sync/test/integration/single_client_managed_user_settings_sync_test.cc
|
| diff --git a/chrome/browser/sync/test/integration/single_client_managed_user_settings_sync_test.cc b/chrome/browser/sync/test/integration/single_client_managed_user_settings_sync_test.cc
|
| index 460f346c3f6817bd43b0e44b547c9aac0e2d95cd..51b25fd7bc330a1a98d578b0b4d2b0b3db600a12 100644
|
| --- a/chrome/browser/sync/test/integration/single_client_managed_user_settings_sync_test.cc
|
| +++ b/chrome/browser/sync/test/integration/single_client_managed_user_settings_sync_test.cc
|
| @@ -4,15 +4,14 @@
|
|
|
| #include "base/prefs/pref_service.h"
|
| #include "base/values.h"
|
| +#include "chrome/browser/managed_mode/managed_user_constants.h"
|
| #include "chrome/browser/managed_mode/managed_user_service.h"
|
| #include "chrome/browser/managed_mode/managed_user_service_factory.h"
|
| -#include "chrome/browser/policy/managed_mode_policy_provider.h"
|
| -#include "chrome/browser/policy/profile_policy_connector.h"
|
| -#include "chrome/browser/policy/profile_policy_connector_factory.h"
|
| +#include "chrome/browser/managed_mode/managed_user_settings_service.h"
|
| +#include "chrome/browser/managed_mode/managed_user_settings_service_factory.h"
|
| #include "chrome/browser/profiles/profile.h"
|
| #include "chrome/browser/sync/profile_sync_service_harness.h"
|
| #include "chrome/browser/sync/test/integration/sync_test.h"
|
| -#include "policy/policy_constants.h"
|
|
|
| class SingleClientManagedUserSettingsSyncTest : public SyncTest {
|
| public:
|
| @@ -21,7 +20,7 @@ class SingleClientManagedUserSettingsSyncTest : public SyncTest {
|
| virtual ~SingleClientManagedUserSettingsSyncTest() {}
|
| };
|
|
|
| -// TODO(pavely): Fix this test.
|
| +// TODO(pavely): Fix this test. See also: http://crbug.com/279307
|
| IN_PROC_BROWSER_TEST_F(SingleClientManagedUserSettingsSyncTest,
|
| DISABLED_Sanity) {
|
| ASSERT_TRUE(SetupClients());
|
| @@ -31,16 +30,14 @@ IN_PROC_BROWSER_TEST_F(SingleClientManagedUserSettingsSyncTest,
|
| // Managed users are prohibited from signing into the browser. Currently
|
| // that means they're also unable to sync anything, so override that for
|
| // this test.
|
| - // TODO(pamg): Remove this override (and the several #includes it requires)
|
| - // once sync and signin are properly separated for managed users. See
|
| - // http://crbug.com/239785.
|
| - policy::ProfilePolicyConnector* connector =
|
| - policy::ProfilePolicyConnectorFactory::GetForProfile(profile);
|
| - policy::ManagedModePolicyProvider* policy_provider =
|
| - connector->managed_mode_policy_provider();
|
| + // TODO(pamg): Remove this override (and the managed user setting it
|
| + // requires) once sync and signin are properly separated for managed users.
|
| + // See http://crbug.com/239785.
|
| + ManagedUserSettingsService* settings_service =
|
| + ManagedUserSettingsServiceFactory::GetForProfile(profile);
|
| scoped_ptr<base::Value> allow_signin(new base::FundamentalValue(true));
|
| - policy_provider->SetLocalPolicyForTesting(policy::key::kSigninAllowed,
|
| - allow_signin.Pass());
|
| + settings_service->SetLocalSettingForTesting(managed_users::kSigninAllowed,
|
| + allow_signin.Pass());
|
|
|
| // The user should not be signed in.
|
| std::string username;
|
|
|