Set the Flywheel promo rate for Android One devices to be 100%.

chrome/browser/io_thread.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/io_thread.h"
 
 #include <vector>
 
 #include "base/base64.h"
 #include "base/bind.h"
@@ skipping 87 matching lines @@
 #endif
 
 #if defined(ENABLE_EXTENSIONS)
 #include "chrome/browser/extensions/event_router_forwarder.h"
 #endif
 
 #if defined(USE_NSS) || defined(OS_IOS)
 #include "net/ocsp/nss_ocsp.h"
 #endif
 
+#if defined(OS_ANDROID)
+#include "base/android/build_info.h"
+#endif
+
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/net/cert_verify_proc_chromeos.h"
 #include "chromeos/network/host_resolver_impl_chromeos.h"
 #endif
 
 using content::BrowserThread;
 
 class SafeBrowsingURLRequestContext;
 
 // The IOThread object must outlive any tasks posted to the IO thread before the
@@ skipping 1046 matching lines @@
       data_reduction_proxy::DataReductionProxyParams::kFallbackAllowed |
       data_reduction_proxy::DataReductionProxyParams::kAlternativeAllowed;
   if (data_reduction_proxy::DataReductionProxyParams::
       IsIncludedInPromoFieldTrial()) {
     flags |= data_reduction_proxy::DataReductionProxyParams::kPromoAllowed;
   }
   if (data_reduction_proxy::DataReductionProxyParams::
       IsIncludedInHoldbackFieldTrial()) {
     flags |= data_reduction_proxy::DataReductionProxyParams::kHoldback;
   }
+#if defined(OS_ANDROID)
+  if (data_reduction_proxy::DataReductionProxyParams::
+      IsIncludedInAndroidOnePromoFieldTrial(
+          base::android::BuildInfo::GetInstance()->android_build_fp())) {
+    flags |= data_reduction_proxy::DataReductionProxyParams::kPromoAllowed;

[mmenke, 2014/12/17 16:32:50]

We already have
data_reduction_proxy::DataReductionProxyParams::IsIncludedInPromoFieldTrial().

Calling into DRP code multiple times to check for different field trials seems
weird.  And then we also have another field trial check just above...

Can we just merge these into one method?  Or since DRP already knows about these
field trials anyways, put the checks in DataReductionProxyParams's constructor?

[jeremyim, 2014/12/17 17:52:49]

This will get cleaned up/refactored in bengr's CL
https://codereview.chromium.org/778463002/

+  }
+#endif
   globals_->data_reduction_proxy_params.reset(
       new data_reduction_proxy::DataReductionProxyParams(flags));
   globals_->data_reduction_proxy_auth_request_handler.reset(
       new data_reduction_proxy::DataReductionProxyAuthRequestHandler(
           DataReductionProxyChromeSettings::GetClient(),
           globals_->data_reduction_proxy_params.get(),
           BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO)));
   globals_->data_reduction_proxy_delegate.reset(
       new data_reduction_proxy::DataReductionProxyDelegate(
           globals_->data_reduction_proxy_auth_request_handler.get(),
@@ skipping 223 matching lines @@
   net::QuicVersionVector supported_versions = net::QuicSupportedVersions();
   for (size_t i = 0; i < supported_versions.size(); ++i) {
     net::QuicVersion version = supported_versions[i];
     if (net::QuicVersionToString(version) == quic_version) {
       return version;
     }
   }
 
   return net::QUIC_VERSION_UNSUPPORTED;
 }