policy: Remove UserCloudPolicyManagerFactory's dependency on Profile.

chrome/browser/ui/webui/policy_ui.cc

Index: chrome/browser/ui/webui/policy_ui.cc
diff --git a/chrome/browser/ui/webui/policy_ui.cc b/chrome/browser/ui/webui/policy_ui.cc
index e2e833cac503f97f558584537e42b4b7d78bfd4b..28a665c62d7fde1530c2a484a40966e1c6e36372 100644
--- a/chrome/browser/ui/webui/policy_ui.cc
+++ b/chrome/browser/ui/webui/policy_ui.cc
@@ -56,6 +56,7 @@
 #else
 #include "chrome/browser/policy/cloud/user_cloud_policy_manager.h"
 #include "chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h"
+#include "content/public/browser/web_contents.h"
 #endif
 
 #if !defined(OS_ANDROID) && !defined(OS_IOS)
@@ -488,8 +489,8 @@ void PolicyUIHandler::RegisterMessages() {
   }
 #else
   policy::UserCloudPolicyManager* user_cloud_policy_manager =
-      policy::UserCloudPolicyManagerFactory::GetForProfile(
-          Profile::FromWebUI(web_ui()));
+      policy::UserCloudPolicyManagerFactory::GetForBrowserContext(
+          web_ui()->GetWebContents()->GetBrowserContext());

[Joao da Silva, 2013/11/04 12:57:45]

You could keep the Profile::FromWebUI arg here

[pneubeck (no reviews), 2013/11/04 13:01:43]

Yes I could, but it's an unnecessary indirection through Profile and dependency
on Profile, IMO.

For the record, this is the implementation of Profile::FromWebUI:
static_cast<Profile*>(web_ui->GetWebContents()->GetBrowserContext());

   if (user_cloud_policy_manager) {
     user_status_provider_.reset(
         new UserPolicyStatusProvider(user_cloud_policy_manager->core()));