Index: chrome/browser/policy/profile_policy_connector.cc |
diff --git a/chrome/browser/policy/profile_policy_connector.cc b/chrome/browser/policy/profile_policy_connector.cc |
index ba88c996e2264e736612ec0b851b8d557ebd8358..6f6af5eadb48917f4732539e76c6181327f98ec5 100644 |
--- a/chrome/browser/policy/profile_policy_connector.cc |
+++ b/chrome/browser/policy/profile_policy_connector.cc |
@@ -11,9 +11,11 @@ |
#include "chrome/browser/policy/browser_policy_connector.h" |
#include "chrome/browser/policy/cloud/cloud_policy_manager.h" |
#include "chrome/browser/policy/configuration_policy_provider.h" |
-#include "chrome/browser/policy/policy_service.h" |
+#include "chrome/browser/policy/forwarding_policy_provider.h" |
+#include "chrome/browser/policy/policy_service_impl.h" |
#include "chrome/browser/policy/schema_registry_service.h" |
#include "chrome/browser/policy/schema_registry_service_factory.h" |
+#include "chrome/browser/profiles/profile.h" |
#if defined(OS_CHROMEOS) |
#include "base/bind.h" |
@@ -22,7 +24,6 @@ |
#include "chrome/browser/chromeos/login/user_manager.h" |
#include "chrome/browser/chromeos/policy/device_local_account_policy_provider.h" |
#include "chrome/browser/chromeos/policy/login_profile_policy_provider.h" |
-#include "chrome/browser/policy/policy_service.h" |
#include "chrome/common/pref_names.h" |
#endif |
@@ -44,12 +45,24 @@ void ProfilePolicyConnector::Init( |
const chromeos::User* user, |
#endif |
CloudPolicyManager* user_cloud_policy_manager) { |
- BrowserPolicyConnector* connector = |
- g_browser_process->browser_policy_connector(); |
// |providers| contains a list of the policy providers available for the |
- // PolicyService of this connector. |
+ // PolicyService of this connector, in decreasing order of priority. |
+ // |
+ // Note: all the providers appended to this vector must eventually become |
+ // initialized for every policy domain, otherwise some subsystems will never |
+ // use the policies exposed by the PolicyService! |
+ // The default ConfigurationPolicyProvider::IsInitializationComplete() |
+ // result is true, so take care if a provider overrides that. |
std::vector<ConfigurationPolicyProvider*> providers; |
+ BrowserPolicyConnector* connector = |
+ g_browser_process->browser_policy_connector(); |
+ forwarding_policy_provider_.reset( |
+ new ForwardingPolicyProvider(connector->GetPlatformProvider())); |
+ forwarding_policy_provider_->Init( |
+ SchemaRegistryServiceFactory::GetForContext(profile_)); |
+ providers.push_back(forwarding_policy_provider_.get()); |
+ |
if (user_cloud_policy_manager) |
providers.push_back(user_cloud_policy_manager); |
@@ -70,7 +83,7 @@ void ProfilePolicyConnector::Init( |
providers.push_back(special_user_policy_provider_.get()); |
#endif |
- policy_service_ = connector->CreatePolicyService(providers); |
+ policy_service_.reset(new PolicyServiceImpl(providers)); |
#if defined(OS_CHROMEOS) |
if (is_primary_user_) { |
@@ -93,6 +106,8 @@ void ProfilePolicyConnector::Shutdown() { |
if (special_user_policy_provider_) |
special_user_policy_provider_->Shutdown(); |
#endif |
+ if (forwarding_policy_provider_) |
+ forwarding_policy_provider_->Shutdown(); |
} |
#if defined(OS_CHROMEOS) |