Move user cloud policy to BrowserProcess.

chrome/browser/browser_process_impl.cc

Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 958f805d366558b7c2862123cf10dd49485b3bc8..fedfc8b0f4515be610d580744818ef46ce2e2cb9 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -43,6 +43,9 @@
 #include "chrome/browser/platform_util.h"
 #include "chrome/browser/plugin_data_remover.h"
 #include "chrome/browser/policy/browser_policy_connector.h"
+#include "chrome/browser/policy/profile_policy_connector.h"

[Mattias Nissler (ping if slow), 2011/05/26 10:20:20]

alphabetize

[sfeuz, 2011/05/31 07:32:31]

Done.

+#include "chrome/browser/policy/cloud_policy_cache_base.h"
+#include "chrome/browser/policy/configuration_policy_pref_store.h"
 #include "chrome/browser/prefs/browser_prefs.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/printing/background_printing_manager.h"
@@ -126,6 +129,9 @@ BrowserProcessImpl::BrowserProcessImpl(const CommandLine& command_line)
       created_devtools_manager_(false),
       created_sidebar_manager_(false),
       created_browser_policy_connector_(false),
+      created_user_policy_connector_(false),
+      created_managed_cloud_policy_provider_(false),
+      created_recommended_cloud_policy_provider_(false),
       created_notification_ui_manager_(false),
       created_safe_browsing_detection_service_(false),
       module_ref_count_(0),
@@ -224,6 +230,9 @@ BrowserProcessImpl::~BrowserProcessImpl() {
   // The policy providers managed by |browser_policy_connector_| need to shut
   // down while the IO and FILE threads are still alive.
   browser_policy_connector_.reset();
+  user_policy_connector_.reset();
+  managed_cloud_policy_provider_.reset();
+  recommended_cloud_policy_provider_.reset();
 
 #if defined(USE_X11)
   // The IO thread must outlive the BACKGROUND_X11 thread.
@@ -530,6 +539,69 @@ policy::BrowserPolicyConnector* BrowserProcessImpl::browser_policy_connector() {
   return browser_policy_connector_.get();
 }
 
+policy::ProfilePolicyConnector* BrowserProcessImpl::user_policy_connector() {
+  DCHECK(CalledOnValidThread());
+  if (!created_user_policy_connector_) {
+    DCHECK(user_policy_connector_.get() == NULL);
+    created_user_policy_connector_ = true;
+    user_policy_connector_.reset(policy::ProfilePolicyConnector::Create());
+  }
+  return user_policy_connector_.get();
+}
+
+policy::ConfigurationPolicyProvider*
+    BrowserProcessImpl::managed_cloud_policy_provider() {
+  DCHECK(CalledOnValidThread());
+  if (!created_managed_cloud_policy_provider_) {
+    DCHECK(!managed_cloud_policy_provider_.get());
+    created_managed_cloud_policy_provider_ = true;
+    // Combining UserPolicy and DevicePolicy.

[Mattias Nissler (ping if slow), 2011/05/26 10:20:20]

The spelling of "UserPolicy" and "DevicePolicy" suggests they are type names.
Let's just write "user policy" and "device policy".

[sfeuz, 2011/05/31 07:32:31]

Done.

+    managed_cloud_policy_provider_.reset(
+        new policy::CombiningCloudPolicyProvider(
+             policy::ConfigurationPolicyPrefStore::

[Mattias Nissler (ping if slow), 2011/05/26 10:20:20]

Indentation

[sfeuz, 2011/05/31 07:32:31]

Done.

+             GetChromePolicyDefinitionList()));
+    // UserPolicy taking precedence.
+    policy::CloudPolicyProvider* user_policy =
+        user_policy_connector()->GetManagedCloudProvider();
+    if (user_policy)
+      managed_cloud_policy_provider_.get()->AddCloudPolicyProvider(user_policy);
+
+    policy::CloudPolicyProvider* device_policy =
+        browser_policy_connector()->GetManagedCloudProvider();
+    if (device_policy)

[Mattias Nissler (ping if slow), 2011/05/26 10:20:20]

Need curlies for multi-line conditional blocks

[sfeuz, 2011/05/31 07:32:31]

Done.

+      managed_cloud_policy_provider_.get()->
+        AddCloudPolicyProvider(device_policy);

[Mattias Nissler (ping if slow), 2011/05/26 10:20:20]

Indentation

[sfeuz, 2011/05/31 07:32:31]

Done.

+  }
+  return managed_cloud_policy_provider_.get();
+}
+
+policy::ConfigurationPolicyProvider*
+    BrowserProcessImpl::recommended_cloud_policy_provider() {
+  DCHECK(CalledOnValidThread());
+  if (!created_recommended_cloud_policy_provider_) {
+    DCHECK(!recommended_cloud_policy_provider_.get());
+    created_recommended_cloud_policy_provider_ = true;
+    // Combining UserPolicy and DevicePolicy.
+    recommended_cloud_policy_provider_.reset(
+        new policy::CombiningCloudPolicyProvider(
+            policy::ConfigurationPolicyPrefStore::
+            GetChromePolicyDefinitionList()));
+    // UserPolicy taking precedence.
+    policy::CloudPolicyProvider* user_policy =
+        user_policy_connector()->GetRecommendedCloudProvider();
+    if (user_policy)
+      recommended_cloud_policy_provider_.get()->
+          AddCloudPolicyProvider(user_policy);
+
+    policy::CloudPolicyProvider* device_policy =
+        browser_policy_connector()->GetRecommendedCloudProvider();
+    if (device_policy)
+      recommended_cloud_policy_provider_.get()->
+          AddCloudPolicyProvider(device_policy);
+  }
+  return recommended_cloud_policy_provider_.get();
+}
+
 IconManager* BrowserProcessImpl::icon_manager() {
   DCHECK(CalledOnValidThread());
   if (!created_icon_manager_)