Respect QuicAllowed policy for new streams

chrome/browser/safe_browsing/safe_browsing_service.cc

Index: chrome/browser/safe_browsing/safe_browsing_service.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_service.cc b/chrome/browser/safe_browsing/safe_browsing_service.cc
index 151e73292b06627d1001ce35002e3a5e2a8ead6c..91d23bf431a283def261c735c28bbad2561459e2 100644
--- a/chrome/browser/safe_browsing/safe_browsing_service.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_service.cc
@@ -139,6 +139,9 @@ class SafeBrowsingURLRequestContextGetter
   // true.
   void ServiceShuttingDown();
 
+  void UpdateNetParamsOnIOThread(
+      net::HttpNetworkSession::ParamsUpdate params_update);
+
  protected:
   ~SafeBrowsingURLRequestContextGetter() override;
 
@@ -239,6 +242,16 @@ void SafeBrowsingURLRequestContextGetter::ServiceShuttingDown() {
   safe_browsing_request_context_.reset();
 }
 
+void SafeBrowsingURLRequestContextGetter::UpdateNetParamsOnIOThread(
+    net::HttpNetworkSession::ParamsUpdate params_update) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+
+  if (!http_network_session_)

[Nathan Parker, 2017/01/05 17:30:18]

nit: could be
if (http_network_session_)
  http_network_session_->...

[pmarko, 2017/01/08 20:51:48]

Done.

+    return;
+
+  http_network_session_->UpdateParams(params_update);
+}
+
 SafeBrowsingURLRequestContextGetter::~SafeBrowsingURLRequestContextGetter() {}
 
 // static
@@ -392,6 +405,13 @@ SafeBrowsingService::url_request_context() {
   return url_request_context_getter_;
 }
 
+void SafeBrowsingService::UpdateNetParamsOnIOThread(
+    net::HttpNetworkSession::ParamsUpdate params_update) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);

[mmenke, 2017/01/06 16:18:43]

Suggest making this method be called on the UI thread, and do the PostTask
itself (And obviously be renamed).  The fact that it lives behind the scenes on
the IO thread doesn't seem like something NetPrefObserver should know or care
about.

[pmarko, 2017/01/08 20:51:48]

Done.

+
+  url_request_context_getter_->UpdateNetParamsOnIOThread(params_update);
+}
+
 // TODO(ntfschr): componentize this once BaseSafeBrowsingUIManager contains a
 // SafeBrowsingService
 const scoped_refptr<SafeBrowsingUIManager>&