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

Unified Diff: net/quic/test_tools/crypto_test_utils.cc

Issue 218923002: Merge internal change: 63891842 - QuicServerId changes (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: net/quic/test_tools/crypto_test_utils.cc
diff --git a/net/quic/test_tools/crypto_test_utils.cc b/net/quic/test_tools/crypto_test_utils.cc
index 45fc5a02e8b0db417acfb40ecdc0dc2cb1808c9b..082e78e2f83fcd456156fdbc466d36ff6e5ba142 100644
--- a/net/quic/test_tools/crypto_test_utils.cc
+++ b/net/quic/test_tools/crypto_test_utils.cc
@@ -15,7 +15,7 @@
#include "net/quic/quic_crypto_client_stream.h"
#include "net/quic/quic_crypto_server_stream.h"
#include "net/quic/quic_crypto_stream.h"
-#include "net/quic/quic_session_key.h"
+#include "net/quic/quic_server_id.h"
#include "net/quic/test_tools/quic_connection_peer.h"
#include "net/quic/test_tools/quic_test_utils.h"
#include "net/quic/test_tools/simple_quic_framer.h"
@@ -41,9 +41,7 @@ class CryptoFramerVisitor : public CryptoFramerVisitorInterface {
: error_(false) {
}
- virtual void OnError(CryptoFramer* framer) OVERRIDE {
- error_ = true;
- }
+ virtual void OnError(CryptoFramer* framer) OVERRIDE { error_ = true; }
virtual void OnHandshakeMessage(
const CryptoHandshakeMessage& message) OVERRIDE {
@@ -179,9 +177,9 @@ int CryptoTestUtils::HandshakeWithFakeClient(
if (options.channel_id_enabled) {
crypto_config.SetChannelIDSigner(ChannelIDSignerForTesting());
}
- QuicSessionKey server_key(kServerHostname, kServerPort, false,
- kPrivacyModeDisabled);
- QuicCryptoClientStream client(server_key, &client_session, NULL,
+ QuicServerId server_id(kServerHostname, kServerPort, false,
+ kPrivacyModeDisabled);
+ QuicCryptoClientStream client(server_id, &client_session, NULL,
&crypto_config);
client_session.SetCryptoStream(&client);
@@ -193,9 +191,9 @@ int CryptoTestUtils::HandshakeWithFakeClient(
CompareClientAndServerKeys(&client, server);
if (options.channel_id_enabled) {
- EXPECT_EQ(crypto_config.channel_id_signer()->GetKeyForHostname(
- kServerHostname),
- server->crypto_negotiated_params().channel_id);
+ EXPECT_EQ(
+ crypto_config.channel_id_signer()->GetKeyForHostname(kServerHostname),
+ server->crypto_negotiated_params().channel_id);
}
return client.num_sent_client_hellos();
@@ -494,7 +492,7 @@ CryptoHandshakeMessage CryptoTestUtils::BuildMessage(const char* message_tag,
valuestr++;
len--;
- CHECK(len % 2 == 0);
+ CHECK_EQ(0u, len % 2);
scoped_ptr<uint8[]> buf(new uint8[len/2]);
for (size_t i = 0; i < len/2; i++) {

Powered by Google App Engine
This is Rietveld 408576698