Enable QUIC for proxies based on Finch config and command line switch.

chrome/browser/profiles/profile_impl_io_data.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/profiles/profile_impl_io_data.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 441 matching lines @@
 
   ApplyProfileParamsToContext(main_context);
 
   if (http_server_properties_manager_)
     http_server_properties_manager_->InitializeOnNetworkThread();
 
   main_context->set_transport_security_state(transport_security_state());
 
   main_context->set_net_log(io_thread->net_log());
 
-  data_reduction_proxy_io_data()->Init();
+  data_reduction_proxy_io_data()->Init(
+      IOThread::ShouldEnableQuicForProxies() &&
+      data_reduction_proxy::DataReductionProxyParams::
+      IsIncludedInQuicFieldTrial());

[mmenke, 2015/02/13 17:45:22]

The extra ShouldEnableQuicForProxies check seems weird, since the second check
being true forces the first to be true...

Suggest putting all this logic in IOThread, and setting another IOThreadGlobal
based on the result (allow_quic_for_data_reduction_proxy or somesuch).  That
would centralize all chrome-side logic on the topic, and we could deal with all
the cases there in one place.

And then here you have:

bool allow_quic_for_data_reduction_proxy = false;
io_thread_globals->allow_quic_for_data_reduction_proxy.CopyToIfSet(&allow_quic_for_data_reduction_proxy);
data_reduction_proxy_io_data()->Init(allow_quic_for_data_reduction_proxy);

[tbansal1, 2015/02/13 19:54:29]

Done.

 
   network_delegate_ = data_reduction_proxy_io_data()->CreateNetworkDelegate(
       chrome_network_delegate.Pass(), true).Pass();
 
   main_context->set_network_delegate(network_delegate_.get());
 
   main_context->set_http_server_properties(http_server_properties());
 
   main_context->set_host_resolver(
       io_thread_globals->host_resolver.get());
@@ skipping 376 matching lines @@
     const base::Closure& completion) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   DCHECK(initialized());
 
   DCHECK(transport_security_state());
   // Completes synchronously.
   transport_security_state()->DeleteAllDynamicDataSince(time);
   DCHECK(http_server_properties_manager_);
   http_server_properties_manager_->Clear(completion);
 }