diff options
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/browser/net/ssl_config_service_manager_pref.cc | 9 | ||||
-rw-r--r-- | chrome/browser/net/ssl_config_service_manager_pref_unittest.cc | 24 | ||||
-rw-r--r-- | chrome/browser/prefs/command_line_pref_store.cc | 2 | ||||
-rw-r--r-- | chrome/common/chrome_switches.cc | 7 | ||||
-rw-r--r-- | chrome/common/chrome_switches.h | 1 | ||||
-rw-r--r-- | chrome/common/pref_names.cc | 2 | ||||
-rw-r--r-- | chrome/common/pref_names.h | 1 |
7 files changed, 2 insertions, 44 deletions
diff --git a/chrome/browser/net/ssl_config_service_manager_pref.cc b/chrome/browser/net/ssl_config_service_manager_pref.cc index 0aab9a6..706bcb0b 100644 --- a/chrome/browser/net/ssl_config_service_manager_pref.cc +++ b/chrome/browser/net/ssl_config_service_manager_pref.cc @@ -176,7 +176,6 @@ class SSLConfigServiceManagerPref StringPrefMember ssl_version_max_; BooleanPrefMember channel_id_enabled_; BooleanPrefMember ssl_record_splitting_disabled_; - BooleanPrefMember unrestricted_ssl3_fallback_enabled_; // The cached list of disabled SSL cipher suites. std::vector<uint16> disabled_cipher_suites_; @@ -210,10 +209,6 @@ SSLConfigServiceManagerPref::SSLConfigServiceManagerPref( prefs::kEnableOriginBoundCerts, local_state, local_state_callback); ssl_record_splitting_disabled_.Init( prefs::kDisableSSLRecordSplitting, local_state, local_state_callback); - unrestricted_ssl3_fallback_enabled_.Init( - prefs::kEnableUnrestrictedSSL3Fallback, - local_state, - local_state_callback); local_state_change_registrar_.Init(local_state); local_state_change_registrar_.Add( @@ -244,8 +239,6 @@ void SSLConfigServiceManagerPref::RegisterPrefs(PrefRegistrySimple* registry) { default_config.channel_id_enabled); registry->RegisterBooleanPref(prefs::kDisableSSLRecordSplitting, !default_config.false_start_enabled); - registry->RegisterBooleanPref(prefs::kEnableUnrestrictedSSL3Fallback, - default_config.unrestricted_ssl3_fallback_enabled); registry->RegisterListPref(prefs::kCipherSuiteBlacklist); } @@ -304,8 +297,6 @@ void SSLConfigServiceManagerPref::GetSSLConfigFromPrefs( config->channel_id_enabled = channel_id_enabled_.GetValue(); // disabling False Start also happens to disable record splitting. config->false_start_enabled = !ssl_record_splitting_disabled_.GetValue(); - config->unrestricted_ssl3_fallback_enabled = - unrestricted_ssl3_fallback_enabled_.GetValue(); } void SSLConfigServiceManagerPref::OnDisabledCipherSuitesChange( diff --git a/chrome/browser/net/ssl_config_service_manager_pref_unittest.cc b/chrome/browser/net/ssl_config_service_manager_pref_unittest.cc index 2cbfc05..753109c 100644 --- a/chrome/browser/net/ssl_config_service_manager_pref_unittest.cc +++ b/chrome/browser/net/ssl_config_service_manager_pref_unittest.cc @@ -146,11 +146,8 @@ TEST_F(SSLConfigServiceManagerPrefTest, BadDisabledCipherSuites) { EXPECT_EQ(0x0005, config.disabled_cipher_suites[1]); } -// Test that -// * without command-line settings for minimum and maximum SSL versions, -// SSL 3.0 ~ default_version_max() are enabled; -// * without --enable-unrestricted-ssl3-fallback, -// |unrestricted_ssl3_fallback_enabled| is false. +// Test that without command-line settings for minimum and maximum SSL versions, +// SSL 3.0 ~ default_version_max() are enabled. TEST_F(SSLConfigServiceManagerPrefTest, NoCommandLinePrefs) { scoped_refptr<TestingPrefStore> local_state_store(new TestingPrefStore()); @@ -174,13 +171,10 @@ TEST_F(SSLConfigServiceManagerPrefTest, NoCommandLinePrefs) { EXPECT_EQ(net::SSL_PROTOCOL_VERSION_SSL3, ssl_config.version_min); EXPECT_EQ(net::SSLConfigService::default_version_max(), ssl_config.version_max); - EXPECT_FALSE(ssl_config.unrestricted_ssl3_fallback_enabled); // The settings should not be added to the local_state. EXPECT_FALSE(local_state->HasPrefPath(prefs::kSSLVersionMin)); EXPECT_FALSE(local_state->HasPrefPath(prefs::kSSLVersionMax)); - EXPECT_FALSE(local_state->HasPrefPath( - prefs::kEnableUnrestrictedSSL3Fallback)); // Explicitly double-check the settings are not in the preference store. std::string version_min_str; @@ -189,10 +183,6 @@ TEST_F(SSLConfigServiceManagerPrefTest, NoCommandLinePrefs) { &version_min_str)); EXPECT_FALSE(local_state_store->GetString(prefs::kSSLVersionMax, &version_max_str)); - bool unrestricted_ssl3_fallback_enabled; - EXPECT_FALSE(local_state_store->GetBoolean( - prefs::kEnableUnrestrictedSSL3Fallback, - &unrestricted_ssl3_fallback_enabled)); } // Test that command-line settings for minimum and maximum SSL versions are @@ -203,7 +193,6 @@ TEST_F(SSLConfigServiceManagerPrefTest, CommandLinePrefs) { CommandLine command_line(CommandLine::NO_PROGRAM); command_line.AppendSwitchASCII(switches::kSSLVersionMin, "tls1"); command_line.AppendSwitchASCII(switches::kSSLVersionMax, "ssl3"); - command_line.AppendSwitch(switches::kEnableUnrestrictedSSL3Fallback); PrefServiceMockFactory factory; factory.set_user_prefs(local_state_store); @@ -224,7 +213,6 @@ TEST_F(SSLConfigServiceManagerPrefTest, CommandLinePrefs) { // Command-line flags should be respected. EXPECT_EQ(net::SSL_PROTOCOL_VERSION_TLS1, ssl_config.version_min); EXPECT_EQ(net::SSL_PROTOCOL_VERSION_SSL3, ssl_config.version_max); - EXPECT_TRUE(ssl_config.unrestricted_ssl3_fallback_enabled); // Explicitly double-check the settings are not in the preference store. const PrefService::Preference* version_min_pref = @@ -235,18 +223,10 @@ TEST_F(SSLConfigServiceManagerPrefTest, CommandLinePrefs) { local_state->FindPreference(prefs::kSSLVersionMax); EXPECT_FALSE(version_max_pref->IsUserModifiable()); - const PrefService::Preference* ssl3_fallback_pref = - local_state->FindPreference(prefs::kEnableUnrestrictedSSL3Fallback); - EXPECT_FALSE(ssl3_fallback_pref->IsUserModifiable()); - std::string version_min_str; std::string version_max_str; EXPECT_FALSE(local_state_store->GetString(prefs::kSSLVersionMin, &version_min_str)); EXPECT_FALSE(local_state_store->GetString(prefs::kSSLVersionMax, &version_max_str)); - bool unrestricted_ssl3_fallback_enabled; - EXPECT_FALSE(local_state_store->GetBoolean( - prefs::kEnableUnrestrictedSSL3Fallback, - &unrestricted_ssl3_fallback_enabled)); } diff --git a/chrome/browser/prefs/command_line_pref_store.cc b/chrome/browser/prefs/command_line_pref_store.cc index 9cdbd4b..f2d6855 100644 --- a/chrome/browser/prefs/command_line_pref_store.cc +++ b/chrome/browser/prefs/command_line_pref_store.cc @@ -57,8 +57,6 @@ const CommandLinePrefStore::BooleanSwitchToPreferenceMapEntry { switches::kDisableTLSChannelID, prefs::kEnableOriginBoundCerts, false }, { switches::kDisableSSLFalseStart, prefs::kDisableSSLRecordSplitting, true }, - { switches::kEnableUnrestrictedSSL3Fallback, - prefs::kEnableUnrestrictedSSL3Fallback, true }, #if defined(GOOGLE_CHROME_BUILD) { switches::kDisablePrintPreview, prefs::kPrintPreviewDisabled, true }, #else diff --git a/chrome/common/chrome_switches.cc b/chrome/common/chrome_switches.cc index e277754..8c7efc6 100644 --- a/chrome/common/chrome_switches.cc +++ b/chrome/common/chrome_switches.cc @@ -707,13 +707,6 @@ const char kEnableThumbnailRetargeting[] = "enable-thumbnail-retargeting"; // Enables Translate experimental new UX which replaces the infobar. const char kEnableTranslateNewUX[] = "enable-translate-new-ux"; -// Enables unrestricted SSL 3.0 fallback. -// With this switch, SSL 3.0 fallback will be enabled for all sites. -// Without this switch, SSL 3.0 fallback will be disabled for a site -// pinned to the Google pin list (indicating that it is a Google site). -const char kEnableUnrestrictedSSL3Fallback[] = - "enable-unrestricted-ssl3-fallback"; - // Enables Alternate-Protocol when the port is user controlled (> 1024). const char kEnableUserAlternateProtocolPorts[] = "enable-user-controlled-alternate-protocol-ports"; diff --git a/chrome/common/chrome_switches.h b/chrome/common/chrome_switches.h index d0a932e..860b793 100644 --- a/chrome/common/chrome_switches.h +++ b/chrome/common/chrome_switches.h @@ -203,7 +203,6 @@ extern const char kEnableSyncArticles[]; extern const char kEnableSyncSyncedNotifications[]; extern const char kEnableThumbnailRetargeting[]; extern const char kEnableTranslateNewUX[]; -extern const char kEnableUnrestrictedSSL3Fallback[]; extern const char kEnableUserAlternateProtocolPorts[]; extern const char kEnableWatchdog[]; extern const char kEnableWebSocketOverSpdy[]; diff --git a/chrome/common/pref_names.cc b/chrome/common/pref_names.cc index 4e62c2a..b7cd4c4 100644 --- a/chrome/common/pref_names.cc +++ b/chrome/common/pref_names.cc @@ -1357,8 +1357,6 @@ const char kSSLVersionMax[] = "ssl.version_max"; const char kCipherSuiteBlacklist[] = "ssl.cipher_suites.blacklist"; const char kEnableOriginBoundCerts[] = "ssl.origin_bound_certs.enabled"; const char kDisableSSLRecordSplitting[] = "ssl.ssl_record_splitting.disabled"; -const char kEnableUnrestrictedSSL3Fallback[] = - "ssl.unrestricted_ssl3_fallback.enabled"; // A boolean pref of the EULA accepted flag. const char kEulaAccepted[] = "EulaAccepted"; diff --git a/chrome/common/pref_names.h b/chrome/common/pref_names.h index bf6d2d6..04cc692 100644 --- a/chrome/common/pref_names.h +++ b/chrome/common/pref_names.h @@ -423,7 +423,6 @@ extern const char kSSLVersionMax[]; extern const char kCipherSuiteBlacklist[]; extern const char kEnableOriginBoundCerts[]; extern const char kDisableSSLRecordSplitting[]; -extern const char kEnableUnrestrictedSSL3Fallback[]; extern const char kGLVendorString[]; extern const char kGLRendererString[]; |