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

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

Issue 208713004: Move SSLConfig class from ssl_config_service.h to ssl_config.h (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
Index: chrome/browser/net/ssl_config_service_manager_pref.cc
diff --git a/chrome/browser/net/ssl_config_service_manager_pref.cc b/chrome/browser/net/ssl_config_service_manager_pref.cc
index 0aab9a6fa7e5b3c42d6c7a805607578f9b0d8d8d..8d4ed126fc3f98a5658b1a02d9684e2f07be6ac1 100644
--- a/chrome/browser/net/ssl_config_service_manager_pref.cc
+++ b/chrome/browser/net/ssl_config_service_manager_pref.cc
@@ -282,8 +282,6 @@ void SSLConfigServiceManagerPref::GetSSLConfigFromPrefs(
rev_checking_required_local_anchors_.GetValue();
std::string version_min_str = ssl_version_min_.GetValue();
std::string version_max_str = ssl_version_max_.GetValue();
- config->version_min = net::SSLConfigService::default_version_min();
- config->version_max = net::SSLConfigService::default_version_max();
wtc 2014/03/22 01:29:41 IMPORTANT: Are you sure these two lines can be del
Sergey Ulanov 2014/03/22 01:49:00 I think they are not necessary, but it's not relev
uint16 version_min = SSLProtocolVersionFromString(version_min_str);
uint16 version_max = SSLProtocolVersionFromString(version_max_str);
if (version_min) {
« no previous file with comments | « no previous file | chrome/browser/net/ssl_config_service_manager_pref_unittest.cc » ('j') | net/ssl/ssl_config_service.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698