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

Unified Diff: net/socket/ssl_client_socket.cc

Issue 356713005: Rename ServerBoundCert => ChannelID to reflect the current name (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 6 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: net/socket/ssl_client_socket.cc
diff --git a/net/socket/ssl_client_socket.cc b/net/socket/ssl_client_socket.cc
index 4f0dd4212c82d97db90db0693ee5f7e7365d8023..55f6ce0109fbbf6df0167f4875a50cad1ce25696 100644
--- a/net/socket/ssl_client_socket.cc
+++ b/net/socket/ssl_client_socket.cc
@@ -7,7 +7,7 @@
#include "base/metrics/histogram.h"
#include "base/strings/string_util.h"
#include "crypto/ec_private_key.h"
-#include "net/ssl/server_bound_cert_service.h"
+#include "net/ssl/channel_id_service.h"
#include "net/ssl/ssl_config_service.h"
namespace net {
@@ -158,7 +158,7 @@ void SSLClientSocket::set_stapled_ocsp_response_received(
// static
void SSLClientSocket::RecordChannelIDSupport(
- ServerBoundCertService* server_bound_cert_service,
+ ChannelIDService* channel_id_service,
bool negotiated_channel_id,
bool channel_id_enabled,
bool supports_ecc) {
@@ -169,17 +169,17 @@ void SSLClientSocket::RecordChannelIDSupport(
CLIENT_AND_SERVER = 2,
CLIENT_NO_ECC = 3,
CLIENT_BAD_SYSTEM_TIME = 4,
- CLIENT_NO_SERVER_BOUND_CERT_SERVICE = 5,
+ CLIENT_NO_CHANNEL_ID_SERVICE = 5,
DOMAIN_BOUND_CERT_USAGE_MAX
wtc 2014/07/01 19:50:53 DOMAIN_BOUND_CERT_USAGE_MAX => CHANNEL_ID_USAGE_MA
Ryan Hamilton 2014/07/21 19:12:09 Done.
} supported = DISABLED;
if (negotiated_channel_id) {
supported = CLIENT_AND_SERVER;
} else if (channel_id_enabled) {
- if (!server_bound_cert_service)
- supported = CLIENT_NO_SERVER_BOUND_CERT_SERVICE;
+ if (!channel_id_service)
+ supported = CLIENT_NO_CHANNEL_ID_SERVICE;
else if (!supports_ecc)
supported = CLIENT_NO_ECC;
- else if (!server_bound_cert_service->IsSystemTimeValid())
+ else if (!channel_id_service->IsSystemTimeValid())
supported = CLIENT_BAD_SYSTEM_TIME;
else
supported = CLIENT_ONLY;
@@ -191,18 +191,18 @@ void SSLClientSocket::RecordChannelIDSupport(
// static
bool SSLClientSocket::IsChannelIDEnabled(
const SSLConfig& ssl_config,
- ServerBoundCertService* server_bound_cert_service) {
+ ChannelIDService* channel_id_service) {
if (!ssl_config.channel_id_enabled)
return false;
- if (!server_bound_cert_service) {
- DVLOG(1) << "NULL server_bound_cert_service_, not enabling channel ID.";
+ if (!channel_id_service) {
+ DVLOG(1) << "NULL channel_id_service_, not enabling channel ID.";
return false;
}
if (!crypto::ECPrivateKey::IsSupported()) {
DVLOG(1) << "Elliptic Curve not supported, not enabling channel ID.";
return false;
}
- if (!server_bound_cert_service->IsSystemTimeValid()) {
+ if (!channel_id_service->IsSystemTimeValid()) {
DVLOG(1) << "System time is not within the supported range for certificate "
"generation, not enabling channel ID.";
return false;

Powered by Google App Engine
This is Rietveld 408576698