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 166 matching lines @@
           net_log, profile_, profile_->GetPrefs(),
           BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO),
           BrowserThread::GetMessageLoopProxyForThread(
               BrowserThread::UI)).Pass());
 
   DataReductionProxyChromeSettingsFactory::GetForBrowserContext(profile_)->
       InitDataReductionProxySettings(io_data_->data_reduction_proxy_io_data(),
                                      profile_->GetPrefs(),
                                      g_browser_process->local_state(),
                                      profile_->GetRequestContext());
+
+  const base::CommandLine& command_line =
+      *base::CommandLine::ForCurrentProcess();
+  bool quic_enabled_for_proxies =
+      IOThread::ShouldEnableQuicForProxies(command_line,

[bengr, 2015/02/11 23:57:28]

Why does command_line need to be passed in?

[tbansal1, 2015/02/12 02:27:21]

This matches ShouldEnableQuic() header. This helps in testing (e.g., in
io_thread_unittest.cc, command line is overridden).

[bengr, 2015/02/12 17:14:46]

But ProfileImplIOData should not need to know how the decision is made by the
DRP component.

[tbansal1, 2015/02/13 17:10:39]

Done.

+                                           IOThread::QuicTrialGroup());
+
+  DataReductionProxyChromeSettingsFactory::GetForBrowserContext(profile_)
+        ->EnableQUIC(quic_enabled_for_proxies);

[bengr, 2015/02/11 23:57:27]

Put the -> on the previous line and indent 4 from DataReduction...

[tbansal1, 2015/02/12 02:27:21]

Done.

 }
 
 content::ResourceContext*
     ProfileImplIOData::Handle::GetResourceContext() const {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   LazyInitialize();
   return GetResourceContextNoInit();
 }
 
 content::ResourceContext*
@@ skipping 650 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);
 }