Move user cloud policy to BrowserProcess.

chrome/browser/chromeos/login/login_utils.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/login/login_utils.h"
 
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/file_path.h"
@@ skipping 20 matching lines @@
 #include "chrome/browser/chromeos/login/ownership_service.h"
 #include "chrome/browser/chromeos/login/parallel_authenticator.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/proxy_config_service.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/net/chrome_url_request_context.h"
 #include "chrome/browser/net/gaia/token_service.h"
 #include "chrome/browser/net/preconnect.h"
 #include "chrome/browser/net/pref_proxy_config_service.h"
 #include "chrome/browser/plugin_updater.h"
+#include "chrome/browser/policy/profile_policy_connector.h"
 #include "chrome/browser/prefs/pref_member.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/ui/browser_init.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/logging_chrome.h"
 #include "chrome/common/net/gaia/gaia_auth_fetcher.h"
 #include "chrome/common/net/gaia/gaia_constants.h"
@@ skipping 174 matching lines @@
 
   // Switch log file as soon as possible.
   logging::RedirectChromeLogging(*(CommandLine::ForCurrentProcess()));
   btl->AddLoginTimeMarker("LoggingRedirected", false);
 
   username_ = username;
   password_ = password;
   credentials_ = credentials;
   pending_requests_ = pending_requests;
   delegate_ = delegate;
 

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

excess newline

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

Done.

+
   // The default profile will have been changed because the ProfileManager
   // will process the notification that the UserManager sends out.
   ProfileManager::CreateDefaultProfileAsync(this);
 }
 
 void LoginUtilsImpl::OnProfileCreated(Profile* profile) {
   CHECK(profile);
 
+  // Initialize the userpolicy backend.
+  ProfilePolicyConnector* profile_policy_connector =
+      g_browser_process->user_policy_connector();
+  DCHECK(profile_policy_connector);

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

No need for the DCHECK, we crash anyway below.

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

Is that really a reason to remove this DCHECK?

+  profile_policy_connector->Initialize(username_,

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

Once we have the policy wrapper object, we should probably just call a
to-be-implemented g_browser_process->InitializeUserPolicy(username_, profile)
here.

+                                       profile()->GetPath(),
+                                       profile->GetTokenService());
+
   BootTimesLoader* btl = BootTimesLoader::Get();
   btl->AddLoginTimeMarker("UserProfileGotten", false);
 
   bool first_login = !UserManager::Get()->IsKnownUser(username_);
   // Change the proxy configuration service of the default request context to
   // use the preference configuration from the logged-in profile. This ensures
   // that requests done through the default context use the proxy configuration
   // provided by configuration policy.
   //
   // Note: Many of the clients of the default request context should probably be
@@ skipping 340 matching lines @@
   // Mark login host for deletion after browser starts.  This
   // guarantees that the message loop will be referenced by the
   // browser before it is dereferenced by the login host.
   if (login_host) {
     login_host->OnSessionStart();
     login_host = NULL;
   }
 }
 
 }  // namespace chromeos