Moved data reduction proxy initialization logic to ProfileImplIOData

chrome/browser/profiles/profile_impl_io_data.h

Index: chrome/browser/profiles/profile_impl_io_data.h
diff --git a/chrome/browser/profiles/profile_impl_io_data.h b/chrome/browser/profiles/profile_impl_io_data.h
index e821b7008b53b599f403ba173de9548ddbd540a3..1fe761975660b5268bfa31d8aa04d89b1d65876c 100644
--- a/chrome/browser/profiles/profile_impl_io_data.h
+++ b/chrome/browser/profiles/profile_impl_io_data.h
@@ -21,6 +21,14 @@ namespace content {
 class CookieCryptoDelegate;
 }  // namespace content
 
+#if defined(SPDY_PROXY_AUTH_ORIGIN)
+namespace data_reduction_proxy {
+class DataReductionProxyParams;
+class DataReductionProxyUsageStats;
+class DataReductionProxyAuthRequestHandler;
+}
+#endif
+
 namespace domain_reliability {
 class DomainReliabilityMonitor;
 }  // namespace domain_reliability
@@ -60,7 +68,9 @@ class ProfileImplIOData : public ProfileIOData {
                   session_cookie_mode,
               quota::SpecialStoragePolicy* special_storage_policy,
               scoped_ptr<domain_reliability::DomainReliabilityMonitor>
-                  domain_reliability_monitor);
+                  domain_reliability_monitor,
+              const base::Callback<void(bool)>&
+                  data_reduction_proxy_unavailable);
 
     // These Create*ContextGetter() functions are only exposed because the
     // circular relationship between Profile, ProfileIOData::Handle, and the
@@ -230,6 +240,17 @@ class ProfileImplIOData : public ProfileIOData {
   int app_cache_max_size_;
   int app_media_cache_max_size_;
 
+#if defined(SPDY_PROXY_AUTH_ORIGIN)

[willchan no longer on Chromium, 2014/07/28 15:45:23]

I need to think further about the right layering here. All the member variables
are sensitive to destruction ordering issues, so if your destructors reach into
any other state, or the network delegate or something reaches into these members
during shutdown, we could have a bug. It looks like our ordering is starting to
get a little sloppy here =/

[bengr, 2014/07/28 20:29:04]

ok. Let me know what you conclude...

+  mutable scoped_ptr<data_reduction_proxy::DataReductionProxyParams>
+      data_reduction_proxy_params_;
+  mutable scoped_ptr<data_reduction_proxy::DataReductionProxyUsageStats>
+      data_reduction_proxy_usage_stats_;
+  mutable base::Callback<void(bool)> data_reduction_proxy_unavailable_callback_;
+
+  mutable scoped_ptr<data_reduction_proxy::DataReductionProxyAuthRequestHandler>
+      data_reduction_proxy_auth_request_handler_;
+#endif  // defined(SPDY_PROXY_AUTH_ORIGIN)
+
   DISALLOW_COPY_AND_ASSIGN(ProfileImplIOData);
 };