Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1822)

Unified Diff: chrome/browser/profiles/profile_impl_io_data.cc

Issue 903213003: Enable QUIC for proxies based on Finch config and command line switch. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed formatting Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/profiles/profile_impl_io_data.cc
diff --git a/chrome/browser/profiles/profile_impl_io_data.cc b/chrome/browser/profiles/profile_impl_io_data.cc
index e2cb0ec4ef3e10a0c01acdce32227b3e2d5591a7..4bc9f6aeb137683d776867bb775d982237a127db 100644
--- a/chrome/browser/profiles/profile_impl_io_data.cc
+++ b/chrome/browser/profiles/profile_impl_io_data.cc
@@ -184,6 +184,15 @@ void ProfileImplIOData::Handle::Init(
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
bengr 2015/02/12 17:14:46 But ProfileImplIOData should not need to know how
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
tbansal1 2015/02/12 02:27:21 Done.
}
content::ResourceContext*

Powered by Google App Engine
This is Rietveld 408576698