Move user cloud policy to BrowserProcess.

chrome/browser/browser_process.h

 // 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.
 
 // This interface is for managing the global services of the application. Each
 // service is lazily created when requested the first time. The service getters
 // will return NULL if the service is not available, so callers must check for
 // this condition.
 
 #ifndef CHROME_BROWSER_BROWSER_PROCESS_H_
@@ skipping 48 matching lines @@
 }
 
 namespace printing {
 class BackgroundPrintingManager;
 class PrintJobManager;
 class PrintPreviewTabController;
 }
 
 namespace policy {
 class BrowserPolicyConnector;
+class ProfilePolicyConnector;

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

Please also give it a better name. Maybe UserPolicyConnector?

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

Done.

+class ConfigurationPolicyProvider;
 }
 
 namespace ui {
 class Clipboard;
 }
 
 // NOT THREAD SAFE, call only from the main thread.
 // These functions shouldn't return NULL unless otherwise noted.
 class BrowserProcess {
  public:
@@ skipping 68 matching lines @@
 
 #if defined(OS_CHROMEOS)
   // Returns thread for websocket to TCP proxy.
   // TODO(dilmah): remove this thread.  Instead provide this functionality via
   // hooks into websocket bridge layer.
   virtual base::Thread* web_socket_proxy_thread() = 0;
 #endif
 
   virtual policy::BrowserPolicyConnector* browser_policy_connector() = 0;
 
+  virtual policy::ProfilePolicyConnector* user_policy_connector() = 0;
+
+  virtual policy::ConfigurationPolicyProvider*
+      recommended_cloud_policy_provider() = 0;
+
+  virtual policy::ConfigurationPolicyProvider*
+      managed_cloud_policy_provider() = 0;

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

We should wrap all the cloud-policy related stuff in one object in order to not
spam the browser_process namespace.

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

I see.
On the other hand: the cloud policy class-system is already quite complicated
and adding another player will not make it easier.

[Mattias Nissler (ping if slow), 2011/05/31 14:14:19]

It's not another player, but just a wrapper. Move everything you have now in
BrowserProcess into some new class and keep only a single pointer here.

+
   virtual IconManager* icon_manager() = 0;
 
   virtual ThumbnailGenerator* GetThumbnailGenerator() = 0;
 
   virtual AutomationProviderList* InitAutomationProviderList() = 0;
 
   virtual void InitDevToolsHttpProtocolHandler(
       const std::string& ip,
       int port,
       const std::string& frontend_url) = 0;
@@ skipping 75 matching lines @@
 
   // Used for testing plugin data removal at shutdown.
   std::string plugin_data_remover_mime_type_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserProcess);
 };
 
 extern BrowserProcess* g_browser_process;
 
 #endif  // CHROME_BROWSER_BROWSER_PROCESS_H_