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

Unified Diff: net/socket/ssl_client_socket_impl.cc

Issue 2607193002: Adding Short Record Header feature. (Closed)
Patch Set: Change name. Created 4 years 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: net/socket/ssl_client_socket_impl.cc
diff --git a/net/socket/ssl_client_socket_impl.cc b/net/socket/ssl_client_socket_impl.cc
index 97c67521063c9c9ba8d1545d9499aac6baec277f..4e3f71311c7c2a4974e2337bc0f583724f3b8a46 100644
--- a/net/socket/ssl_client_socket_impl.cc
+++ b/net/socket/ssl_client_socket_impl.cc
@@ -234,6 +234,9 @@ bool AreLegacyECDSACiphersEnabled() {
}
#endif
+const base::Feature kShortRecordHeaderFeature{
+ "SSLShortRecordHeader", base::FEATURE_DISABLED_BY_DEFAULT};
davidben 2017/01/02 10:48:10 Should this go in testing/variations/fieldtrial_te
+
} // namespace
class SSLClientSocketImpl::SSLContext {
@@ -289,6 +292,10 @@ class SSLClientSocketImpl::SSLContext {
SSL_CTX_set_grease_enabled(ssl_ctx_.get(), 1);
+ if (base::FeatureList::IsEnabled(kShortRecordHeaderFeature)) {
+ SSL_CTX_set_short_header_enabled(ssl_ctx_.get(), 1);
+ }
+
if (!SSL_CTX_add_client_custom_ext(ssl_ctx_.get(), kTbExtNum,
&TokenBindingAddCallback,
&TokenBindingFreeCallback, nullptr,
« 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