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

Unified Diff: components/ssl_config/ssl_config_service_manager_pref.cc

Issue 2711633004: Disconnect TLS 1.3 from base::FeatureList in M56. (Closed)
Patch Set: fix test Created 3 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
« no previous file with comments | « no previous file | components/ssl_config/ssl_config_service_manager_pref_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/ssl_config/ssl_config_service_manager_pref.cc
diff --git a/components/ssl_config/ssl_config_service_manager_pref.cc b/components/ssl_config/ssl_config_service_manager_pref.cc
index a870ab12a0c4aaf1ca708e6c6fe05656a788963c..46a26c575b8241ef91645f040c273ce1db62ac36 100644
--- a/components/ssl_config/ssl_config_service_manager_pref.cc
+++ b/components/ssl_config/ssl_config_service_manager_pref.cc
@@ -85,10 +85,6 @@ uint16_t SSLProtocolVersionFromString(const std::string& version_str) {
return version;
}
-const base::Feature kTLS13Feature{
- "NegotiateTLS13", base::FEATURE_DISABLED_BY_DEFAULT,
-};
-
} // namespace
////////////////////////////////////////////////////////////////////////////////
@@ -194,12 +190,6 @@ SSLConfigServiceManagerPref::SSLConfigServiceManagerPref(
io_task_runner_(io_task_runner) {
DCHECK(local_state);
- if (base::FeatureList::IsEnabled(kTLS13Feature)) {
- local_state->SetDefaultPrefValue(
- ssl_config::prefs::kSSLVersionMax,
- new base::StringValue(switches::kSSLVersionTLSv13));
- }
-
PrefChangeRegistrar::NamedChangeCallback local_state_callback =
base::Bind(&SSLConfigServiceManagerPref::OnPreferenceChanged,
base::Unretained(this), local_state);
« no previous file with comments | « no previous file | components/ssl_config/ssl_config_service_manager_pref_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698