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

Unified Diff: extensions/browser/api/socket/tls_socket.cc

Issue 1139013002: Completely remove SSLv3 support. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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: extensions/browser/api/socket/tls_socket.cc
diff --git a/extensions/browser/api/socket/tls_socket.cc b/extensions/browser/api/socket/tls_socket.cc
index 9ee15ed91289dd058c3a5f53c091389b9c1b5fb3..4b34c4bbfa504a873f8b5f597d9762387759c6b0 100644
--- a/extensions/browser/api/socket/tls_socket.cc
+++ b/extensions/browser/api/socket/tls_socket.cc
@@ -23,9 +23,7 @@ namespace {
// Returns 0 if the string is invalid.
uint16 SSLProtocolVersionFromString(const std::string& version_str) {
uint16 version = 0; // Invalid.
- if (version_str == "ssl3") {
- version = net::SSL_PROTOCOL_VERSION_SSL3;
- } else if (version_str == "tls1") {
+ if (version_str == "tls1") {
davidben 2015/05/13 20:46:18 NOTE: Unlike the rest of this CL, this part is not
version = net::SSL_PROTOCOL_VERSION_TLS1;
} else if (version_str == "tls1.1") {
version = net::SSL_PROTOCOL_VERSION_TLS1_1;

Powered by Google App Engine
This is Rietveld 408576698