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

Unified Diff: net/base/ssl_config_service_mac_unittest.cc

Issue 4091005: Remove SSL 2.0 support. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Sync with ToT Created 10 years, 1 month 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 | « net/base/ssl_config_service_mac.cc ('k') | net/base/ssl_config_service_win.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/base/ssl_config_service_mac_unittest.cc
===================================================================
--- net/base/ssl_config_service_mac_unittest.cc (revision 67717)
+++ net/base/ssl_config_service_mac_unittest.cc (working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -39,7 +39,6 @@
// Verify that the constructor sets the correct default values.
net::SSLConfig config;
EXPECT_TRUE(config.rev_checking_enabled);
- EXPECT_FALSE(config.ssl2_enabled);
EXPECT_TRUE(config.ssl3_enabled);
EXPECT_TRUE(config.tls1_enabled);
@@ -69,19 +68,6 @@
net::SSLConfigServiceMac::SetRevCheckingEnabled(
config_save.rev_checking_enabled);
- // Test SetSSL2Enabled.
- net::SSLConfigServiceMac::SetSSL2Enabled(true);
- rv = net::SSLConfigServiceMac::GetSSLConfigNow(&config);
- EXPECT_TRUE(rv);
- EXPECT_TRUE(config.ssl2_enabled);
-
- net::SSLConfigServiceMac::SetSSL2Enabled(false);
- rv = net::SSLConfigServiceMac::GetSSLConfigNow(&config);
- EXPECT_TRUE(rv);
- EXPECT_FALSE(config.ssl2_enabled);
-
- net::SSLConfigServiceMac::SetSSL2Enabled(config_save.ssl2_enabled);
-
// Test SetSSL3Enabled.
net::SSLConfigServiceMac::SetSSL3Enabled(true);
rv = net::SSLConfigServiceMac::GetSSLConfigNow(&config);
@@ -147,15 +133,15 @@
EXPECT_TRUE(rv);
net::SSLConfig config;
- net::SSLConfigServiceMac::SetSSL2Enabled(false);
+ net::SSLConfigServiceMac::SetSSL3Enabled(false);
config_service->GetSSLConfigAt(&config, now);
// Add an observer.
SSLConfigServiceMacObserver observer;
config_service->AddObserver(&observer);
- // Toggle SSL2.
- net::SSLConfigServiceMac::SetSSL2Enabled(!config_save.ssl2_enabled);
+ // Toggle SSL3.
+ net::SSLConfigServiceMac::SetSSL3Enabled(!config_save.ssl3_enabled);
config_service->GetSSLConfigAt(&config, later);
// Verify that the observer was notified.
@@ -164,7 +150,7 @@
// Remove the observer.
config_service->RemoveObserver(&observer);
- // Restore the original SSL2 setting.
- net::SSLConfigServiceMac::SetSSL2Enabled(config_save.ssl2_enabled);
+ // Restore the original SSL3 setting.
+ net::SSLConfigServiceMac::SetSSL3Enabled(config_save.ssl3_enabled);
}
« no previous file with comments | « net/base/ssl_config_service_mac.cc ('k') | net/base/ssl_config_service_win.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698