Data Reduction Proxy class ownership updates and Settings cleanup

chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.h

Index: chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.h
diff --git a/chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.h b/chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.h
index 57fbf651ca856c755741eba0a145df0426c780db..2c363dc6070141984175d8250c58bf911fd6ea68 100644
--- a/chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.h
+++ b/chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.h
@@ -5,7 +5,6 @@
 #ifndef CHROME_BROWSER_NET_SPDYPROXY_DATA_REDUCTION_PROXY_CHROME_SETTINGS_H_
 #define CHROME_BROWSER_NET_SPDYPROXY_DATA_REDUCTION_PROXY_CHROME_SETTINGS_H_
 
-#include "base/memory/scoped_ptr.h"
 #include "components/data_reduction_proxy/core/browser/data_reduction_proxy_auth_request_handler.h"
 #include "components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.h"
 #include "components/keyed_service/core/keyed_service.h"
@@ -15,15 +14,9 @@ class PrefService;
 }
 
 namespace data_reduction_proxy {
-class DataReductionProxyConfigurator;
+class DataReductionProxyConfig;
 class DataReductionProxyIOData;
 class DataReductionProxyEventStore;
-class DataReductionProxyParams;
-}
-
-namespace net {
-class NetLog;
-class URLRequestContextGetter;
 }
 
 class PrefService;
@@ -36,19 +29,16 @@ class DataReductionProxyChromeSettings
  public:
   // Constructs a settings object with the given configuration parameters.

[bengr, 2015/02/03 21:51:57]

What parameters?

[jeremyim, 2015/02/04 01:31:20]

Done.

   // Construction and destruction must happen on the UI thread.
-  explicit DataReductionProxyChromeSettings(
-      scoped_ptr<data_reduction_proxy::DataReductionProxyParams> params);
+  DataReductionProxyChromeSettings();

[bengr, 2015/02/03 21:51:57]

I would love to see this renamed as DataReductionProxyUIData, but probably not
in this CL.

Actually, DataSaverUIData would be even better.

[jeremyim, 2015/02/04 01:31:20]

Acknowledged.

 
   // Destructs the settings object.
   ~DataReductionProxyChromeSettings() override;
 
-  // Initialize the settings object with the given configurator, prefs services,
-  // and request context.
+  // Initialize the settings object with the given prefs services, and io_data.
   void InitDataReductionProxySettings(
       data_reduction_proxy::DataReductionProxyIOData* io_data,
       PrefService* profile_prefs,
-      PrefService* local_state_prefs,
-      net::URLRequestContextGetter* request_context);
+      PrefService* local_state_prefs);

[bengr, 2015/02/03 21:51:58]

Do we still need local state?

[jeremyim, 2015/02/04 01:31:20]

Done.

 
   // Gets the client type for the data reduction proxy.
   static data_reduction_proxy::Client GetClient();