policy: Remove UserCloudPolicyManagerFactory's dependency on Profile.

chrome/browser/policy/cloud/user_cloud_policy_store.cc

 // Copyright (c) 2012 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/policy/cloud/user_cloud_policy_store.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/location.h"
 #include "base/task_runner_util.h"
 #include "chrome/browser/policy/proto/cloud/device_management_backend.pb.h"
 #include "chrome/browser/policy/proto/cloud/device_management_local.pb.h"
-#include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/signin_manager.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "policy/policy_constants.h"
 #include "policy/proto/cloud_policy.pb.h"
 
 namespace em = enterprise_management;
 
 namespace policy {
 
 enum PolicyLoadStatus {
@@ skipping 63 matching lines @@
 
   int size = data.size();
   if (file_util::WriteFile(path, data.c_str(), size) != size) {
     DLOG(WARNING) << "Failed to write " << path.value();
   }
 }
 
 }  // namespace
 
 UserCloudPolicyStore::UserCloudPolicyStore(
-    Profile* profile,
+    content::BrowserContext* context,
     const base::FilePath& path,
     scoped_refptr<base::SequencedTaskRunner> background_task_runner)
     : UserCloudPolicyStoreBase(background_task_runner),
       weak_factory_(this),
-      profile_(profile),
+      context_(context),
       backing_file_path_(path) {}
 
 UserCloudPolicyStore::~UserCloudPolicyStore() {}
 
 // static
 scoped_ptr<UserCloudPolicyStore> UserCloudPolicyStore::Create(
-    Profile* profile,
+    content::BrowserContext* context,
     scoped_refptr<base::SequencedTaskRunner> background_task_runner) {
   base::FilePath path =
-      profile->GetPath().Append(kPolicyDir).Append(kPolicyCacheFile);
+      context->GetPath().Append(kPolicyDir).Append(kPolicyCacheFile);
   return make_scoped_ptr(
-      new UserCloudPolicyStore(profile, path, background_task_runner));
+      new UserCloudPolicyStore(context, path, background_task_runner));
 }
 
 void UserCloudPolicyStore::LoadImmediately() {
   DVLOG(1) << "Initiating immediate policy load from disk";
   // Cancel any pending Load/Store/Validate operations.
   weak_factory_.InvalidateWeakPtrs();
   // Load the policy from disk...
   PolicyLoadResult result = LoadPolicyFromDisk(backing_file_path_);
   // ...and install it, reporting success/failure to any observers.
   PolicyLoaded(false, result);
@@ skipping 85 matching lines @@
 }
 
 void UserCloudPolicyStore::Validate(
     scoped_ptr<em::PolicyFetchResponse> policy,
     bool validate_in_background,
     const UserCloudPolicyValidator::CompletionCallback& callback) {
   // Configure the validator.
   scoped_ptr<UserCloudPolicyValidator> validator = CreateValidator(
       policy.Pass(),
       CloudPolicyValidatorBase::TIMESTAMP_NOT_BEFORE);
-  SigninManager* signin = SigninManagerFactory::GetForProfileIfExists(profile_);
+  SigninManager* signin =
+      SigninManagerFactory::GetForBrowserContextIfExists(context_);

[Joao da Silva, 2013/11/04 10:02:55]

This renaming (and the changes to signin_manager_*) shouldn't be needed after
https://codereview.chromium.org/49783006, right?

   if (signin) {
     std::string username = signin->GetAuthenticatedUsername();
     if (username.empty())
       username = signin->GetUsernameForAuthInProgress();
 
     // Validate the username if the user is signed in (or in the process of
     // signing in).
     if (!username.empty())
       validator->ValidateUsername(username);
   }
@@ skipping 24 matching lines @@
   background_task_runner()->PostTask(
       FROM_HERE,
       base::Bind(&StorePolicyToDiskOnBackgroundThread,
                  backing_file_path_, *validator->policy()));
   InstallPolicy(validator->policy_data().Pass(), validator->payload().Pass());
   status_ = STATUS_OK;
   NotifyStoreLoaded();
 }
 
 }  // namespace policy