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

Unified Diff: chrome/browser/net/preconnect.cc

Issue 1850043002: Check whether to enable token binding in preconnect.cc (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/preconnect.cc
diff --git a/chrome/browser/net/preconnect.cc b/chrome/browser/net/preconnect.cc
index 1e920afbfa62b026c14fd5279fff4a7d911ccf9b..cb28b96bcf598f34e545cd357627888872eef051 100644
--- a/chrome/browser/net/preconnect.cc
+++ b/chrome/browser/net/preconnect.cc
@@ -105,6 +105,10 @@ void PreconnectOnIOThread(const GURL& url,
session->ssl_config_service()->GetSSLConfig(&ssl_config);
session->GetAlpnProtos(&ssl_config.alpn_protos);
session->GetNpnProtos(&ssl_config.npn_protos);
+ if (session->params().enable_token_binding &&
+ session->params().channel_id_service) {
+ ssl_config.token_binding_params.push_back(net::TB_PARAM_ECDSAP256);
mattm 2016/04/01 01:27:18 Should this check privacy mode? Should it set ssl_
nharper 2016/04/01 19:33:06 ssl_config.channel_id_enabled is true by default,
+ }
// All preconnects should perform EV certificate verification.
ssl_config.verify_ev_cert = true;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698