Cleanup the UseSharedProxies preference.

chrome/browser/chromeos/proxy_config_service_impl.h

Index: chrome/browser/chromeos/proxy_config_service_impl.h
diff --git a/chrome/browser/chromeos/proxy_config_service_impl.h b/chrome/browser/chromeos/proxy_config_service_impl.h
index c45c4b7412ffac142b7fc96c261bcf45a8dd8027..28375421ca43694657944b08ab6ad7917bbc802d 100644
--- a/chrome/browser/chromeos/proxy_config_service_impl.h
+++ b/chrome/browser/chromeos/proxy_config_service_impl.h
@@ -38,14 +38,20 @@ class ProxyConfigServiceImpl : public PrefProxyConfigTrackerImpl,
  public:
   // ProxyConfigServiceImpl is created in ProxyServiceFactory::
   // CreatePrefProxyConfigTrackerImpl via Profile::GetProxyConfigTracker() for
-  // profile or IOThread constructor for local state and is owned by the
+  // profile or via IOThread constructor for local state and is owned by the
   // respective classes.
   //
-  // The new modified proxy config, together with proxy from prefs if available,
-  // are used to determine the effective proxy config, which is then pushed
-  // through PrefProxyConfigTrackerImpl to ChromeProxyConfigService to the
+  // The user's proxy config, proxy policies and proxy from prefs, are used to
+  // determine the effective proxy config, which is then pushed through
+  // PrefProxyConfigTrackerImpl to ChromeProxyConfigService to the
   // network stack.
-  explicit ProxyConfigServiceImpl(PrefService* pref_service);
+  //
+  // |signin_profile| must be true, if this object is associated to the signin
+  // profile. |profile_prefs| must be NULL if this object is constructed for the
+  // system request context.
+  explicit ProxyConfigServiceImpl(bool signin_profile,
+                                  PrefService* profile_prefs,
+                                  PrefService* local_state_prefs);
   virtual ~ProxyConfigServiceImpl();
 
   // PrefProxyConfigTrackerImpl implementation.
@@ -55,21 +61,14 @@ class ProxyConfigServiceImpl : public PrefProxyConfigTrackerImpl,
   // NetworkStateHandlerObserver implementation.
   virtual void DefaultNetworkChanged(const NetworkState* network) OVERRIDE;
 
-  // Register UseShardProxies preference.
-  static void RegisterPrefs(PrefRegistrySimple* registry);
-  static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry);
-
  protected:
   friend class UIProxyConfigService;
 
-  // Returns value of UseSharedProxies preference if it's not default, else
-  // returns false if user is logged in and true otherwise.
-  static bool GetUseSharedProxies(const PrefService* pref_service);
-
   // Returns true if proxy is to be ignored for this profile and |onc_source|,
   // e.g. this happens if the network is shared and use-shared-proxies is turned
-  // off.
-  static bool IgnoreProxy(const PrefService* pref_service,
+  // off. |profile_prefs| can be NULL for system request context.
+  static bool IgnoreProxy(bool signin_profile,
+                          const PrefService* profile_prefs,
                           const std::string network_profile_path,
                           onc::ONCSource onc_source);
 
@@ -93,6 +92,12 @@ class ProxyConfigServiceImpl : public PrefProxyConfigTrackerImpl,
   // Track changes in UseSharedProxies user preference.
   BooleanPrefMember use_shared_proxies_;
 
+  // Not owned. NULL in the system request context.
+  PrefService* profile_prefs_;
+
+  // Wether this object is associated to the signin profile.

[battre, 2013/07/11 13:31:23]

ni: Whether

[pneubeck (no reviews), 2013/07/12 09:42:30]

Done.

+  bool signin_profile_;
+
   base::WeakPtrFactory<ProxyConfigServiceImpl> pointer_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ProxyConfigServiceImpl);