| Index: chrome/browser/policy/browser_policy_connector.h
|
| diff --git a/chrome/browser/policy/browser_policy_connector.h b/chrome/browser/policy/browser_policy_connector.h
|
| index 2dbd1b4612311085f12b545528d268cd49cbe371..50d4462ff6751d1d0bcdeb4f4040395824be869f 100644
|
| --- a/chrome/browser/policy/browser_policy_connector.h
|
| +++ b/chrome/browser/policy/browser_policy_connector.h
|
| @@ -40,9 +40,16 @@ class BrowserPolicyConnector : public content::NotificationObserver {
|
| TOKEN_TYPE_OAUTH, // An OAuth v2 access token.
|
| };
|
|
|
| - static BrowserPolicyConnector* Create();
|
| + // Builds an uninitialized BrowserPolicyConnector, suitable for testing.
|
| + // Init() should be called to create and start the policy machinery.
|
| + BrowserPolicyConnector();
|
| virtual ~BrowserPolicyConnector();
|
|
|
| + // Creates the policy providers and finalizes the initialization of the
|
| + // connector. This call can be skipped on tests that don't require the full
|
| + // policy system running.
|
| + void Init();
|
| +
|
| ConfigurationPolicyProvider* GetManagedPlatformProvider() const;
|
| ConfigurationPolicyProvider* GetManagedCloudProvider() const;
|
| ConfigurationPolicyProvider* GetRecommendedPlatformProvider() const;
|
| @@ -86,12 +93,12 @@ class BrowserPolicyConnector : public content::NotificationObserver {
|
| // and clears the error flags, so potential retries have a chance to succeed.
|
| void ResetDevicePolicy();
|
|
|
| - // Initiates a policy fetch after a successful device registration.
|
| - void FetchDevicePolicy();
|
| + // Initiates device and user policy fetches, if possible. Pending fetches
|
| + // will be cancelled.
|
| + void FetchCloudPolicy();
|
|
|
| - // Initiates a user policy fetch after a successful device registration. This
|
| - // is only safe to call when a user device token is available.
|
| - void FetchUserPolicy();
|
| + // Refreshes policies on each existing provider.
|
| + void RefreshPolicies();
|
|
|
| // Schedules initialization of the cloud policy backend services, if the
|
| // services are already constructed.
|
| @@ -124,18 +131,6 @@ class BrowserPolicyConnector : public content::NotificationObserver {
|
| const std::string& user_name);
|
|
|
| private:
|
| - friend class ::TestingBrowserProcess;
|
| -
|
| - BrowserPolicyConnector();
|
| -
|
| - // Constructor for tests that allows tests to use fake platform and cloud
|
| - // policy providers instead of using the actual ones.
|
| - BrowserPolicyConnector(
|
| - ConfigurationPolicyProvider* managed_platform_provider,
|
| - ConfigurationPolicyProvider* recommended_platform_provider,
|
| - CloudPolicyProvider* managed_cloud_provider,
|
| - CloudPolicyProvider* recommended_cloud_provider);
|
| -
|
| // content::NotificationObserver method overrides:
|
| virtual void Observe(int type,
|
| const content::NotificationSource& source,
|
| @@ -149,7 +144,6 @@ class BrowserPolicyConnector : public content::NotificationObserver {
|
| // be running.
|
| void InitializeDevicePolicySubsystem();
|
|
|
| - static BrowserPolicyConnector* CreateForTests();
|
| static ConfigurationPolicyProvider* CreateManagedPlatformProvider();
|
| static ConfigurationPolicyProvider* CreateRecommendedPlatformProvider();
|
|
|
|
|