Componentize proxy code from chrome/browser/net

components/proxy_config/pref_proxy_config_tracker_impl.h

 // 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.
 
-#ifndef CHROME_BROWSER_NET_PREF_PROXY_CONFIG_TRACKER_IMPL_H_
-#define CHROME_BROWSER_NET_PREF_PROXY_CONFIG_TRACKER_IMPL_H_
+#ifndef COMPONENTS_PROXY_CONFIG_PREF_PROXY_CONFIG_TRACKER_IMPL_H_
+#define COMPONENTS_PROXY_CONFIG_PREF_PROXY_CONFIG_TRACKER_IMPL_H_
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "base/prefs/pref_change_registrar.h"
-#include "chrome/browser/net/pref_proxy_config_tracker.h"
+#include "base/threading/thread_checker.h"
+#include "components/proxy_config/pref_proxy_config_tracker.h"
 #include "components/proxy_config/proxy_config_dictionary.h"
 #include "net/proxy/proxy_config.h"
 #include "net/proxy/proxy_config_service.h"
 
 class PrefService;
 class PrefRegistrySimple;
 
+namespace base {
+class SingleThreadTaskRunner;
+}
+
 namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
 // A net::ProxyConfigService implementation that applies preference proxy
 // settings (pushed from PrefProxyConfigTrackerImpl) as overrides to the proxy
 // configuration determined by a baseline delegate ProxyConfigService on
 // non-ChromeOS platforms. ChromeOS has its own implementation of overrides in
 // chromeos::ProxyConfigServiceImpl.
-class ChromeProxyConfigService
-    : public net::ProxyConfigService,
-      public net::ProxyConfigService::Observer {
+class ProxyConfigServiceImpl : public net::ProxyConfigService,
+                               public net::ProxyConfigService::Observer {
  public:
   // Takes ownership of the passed |base_service|.
   // GetLatestProxyConfig returns ConfigAvailability::CONFIG_PENDING until
   // UpdateProxyConfig has been called.
-  explicit ChromeProxyConfigService(net::ProxyConfigService* base_service);
-  ~ChromeProxyConfigService() override;
+  explicit ProxyConfigServiceImpl(net::ProxyConfigService* base_service);
+  ~ProxyConfigServiceImpl() override;
 
   // ProxyConfigService implementation:
   void AddObserver(net::ProxyConfigService::Observer* observer) override;
   void RemoveObserver(net::ProxyConfigService::Observer* observer) override;
   ConfigAvailability GetLatestProxyConfig(net::ProxyConfig* config) override;
   void OnLazyPoll() override;
 
   // Method on IO thread that receives the preference proxy settings pushed from
   // PrefProxyConfigTrackerImpl.
   void UpdateProxyConfig(ProxyPrefs::ConfigState config_state,
@@ skipping 17 matching lines @@
   // Configuration as defined by prefs.
   net::ProxyConfig pref_config_;
 
   // Flag that indicates that a PrefProxyConfigTracker needs to inform us
   // about a proxy configuration before we may return any configuration.
   bool pref_config_read_pending_;
 
   // Indicates whether the base service registration is done.
   bool registered_observer_;
 
-  DISALLOW_COPY_AND_ASSIGN(ChromeProxyConfigService);
+  base::ThreadChecker thread_checker_;
+
+  DISALLOW_COPY_AND_ASSIGN(ProxyConfigServiceImpl);
 };
 
 // A class that tracks proxy preferences. It translates the configuration
 // to net::ProxyConfig and pushes the result over to the IO thread for
-// ChromeProxyConfigService::UpdateProxyConfig to use.
+// ProxyConfigServiceImpl::UpdateProxyConfig to use.
 class PrefProxyConfigTrackerImpl : public PrefProxyConfigTracker {
  public:
-  explicit PrefProxyConfigTrackerImpl(PrefService* pref_service);
+  PrefProxyConfigTrackerImpl(
+      PrefService* pref_service,
+      scoped_refptr<base::SingleThreadTaskRunner> io_task_runner);
   ~PrefProxyConfigTrackerImpl() override;
 
   // PrefProxyConfigTracker implementation:
   scoped_ptr<net::ProxyConfigService> CreateTrackingProxyConfigService(
       scoped_ptr<net::ProxyConfigService> base_service) override;
 
   // Notifies the tracker that the pref service passed upon construction is
   // about to go away. This must be called from the UI thread.
   void DetachFromPrefService() override;
 
@@ skipping 53 matching lines @@
 
  private:
   // Tracks configuration state. |pref_config_| is valid only if |config_state_|
   // is not CONFIG_UNSET.
   ProxyPrefs::ConfigState config_state_;
 
   // Configuration as defined by prefs.
   net::ProxyConfig pref_config_;
 
   PrefService* pref_service_;
-  ChromeProxyConfigService* chrome_proxy_config_service_;  // Weak ptr.
+  ProxyConfigServiceImpl* chrome_proxy_config_service_;  // Weak ptr.

[droger, 2015/08/19 09:18:45]

nit: rename this to:
proxy_config_service_impl_

[Abhishek, 2015/08/19 10:10:29]

Done.

   bool update_pending_;  // True if config has not been pushed to network stack.
   PrefChangeRegistrar proxy_prefs_;
 
+  scoped_refptr<base::SingleThreadTaskRunner> io_task_runner_;
+
+  base::ThreadChecker thread_checker_;
+
   DISALLOW_COPY_AND_ASSIGN(PrefProxyConfigTrackerImpl);
 };
 
-#endif  // CHROME_BROWSER_NET_PREF_PROXY_CONFIG_TRACKER_IMPL_H_
+#endif  // COMPONENTS_PROXY_CONFIG_PREF_PROXY_CONFIG_TRACKER_IMPL_H_