Add data reduction functionality to all platforms.

chrome/browser/net/chrome_network_delegate.cc

 // 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.
 
 #include "chrome/browser/net/chrome_network_delegate.h"
 
 #include <stdlib.h>
 
 #include <vector>
 
@@ skipping 116 matching lines @@
 
   PrefService* prefs = g_browser_process->local_state();
   if (!prefs)
     return;
 
   // Ignore off-the-record data.
   if (!g_browser_process->profile_manager()->IsValidProfile(profile) ||
       profile->IsOffTheRecord()) {
     return;
   }
-#if defined(OS_ANDROID) && defined(SPDY_PROXY_AUTH_ORIGIN)
-  // If Android ever goes multi profile, the profile should be passed so that
-  // the browser preference will be taken.
   bool with_data_reduction_proxy_enabled =
-      ProfileManager::GetActiveUserProfile()->GetPrefs()->GetBoolean(
+      profile->GetPrefs()->GetBoolean(
           data_reduction_proxy::prefs::kDataReductionProxyEnabled);

[mmenke, 2014/09/10 15:16:48]

Could we just move this into UpdateContentLengthPrefs?  My general feeling is
that, with the possible exception of actually configuring the pref, it makes the
most sense to either only read the pref in the component, or only read it in the
component, rather than to split the responsibility of checking it between the
two.

Does mean "UpdateContentLengthPrefs" has to distingush between local state
prefs, and per user prefs, which isn't lovely...but we have to do that, here,
anyways.

[Not at Google. Contact bengr, 2014/09/10 21:38:34]

Done.

In the future we want to move off of local state completely and only use the
profile specific prefs.

-#else
-  bool with_data_reduction_proxy_enabled = false;
-#endif
 
   data_reduction_proxy::UpdateContentLengthPrefs(received_content_length,
                                          original_content_length,
                                          with_data_reduction_proxy_enabled,
                                          request_type, prefs);

[mmenke, 2014/09/10 15:16:48]

While you're here, mind fixing the indent here?

[Not at Google. Contact bengr, 2014/09/10 21:38:34]

Done.

 }
 
 void StoreAccumulatedContentLength(
     int received_content_length,
     int original_content_length,
     data_reduction_proxy::DataReductionProxyRequestType request_type,
     Profile* profile) {
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
       base::Bind(&UpdateContentLengthPrefs,
                  received_content_length, original_content_length,
@@ skipping 660 matching lines @@
     data_reduction_proxy::DataReductionProxyRequestType request_type) {
   DCHECK_GE(received_content_length, 0);
   DCHECK_GE(original_content_length, 0);
   StoreAccumulatedContentLength(received_content_length,
                                 original_content_length,
                                 request_type,
                                 reinterpret_cast<Profile*>(profile_));
   received_content_length_ += received_content_length;
   original_content_length_ += original_content_length;
 }