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

Unified Diff: net/quic/core/crypto/quic_crypto_client_config_test.cc

Issue 2589983002: Create a QUIC wrapper around scoped_refptr. (Closed)
Patch Set: rm = nullptr Created 4 years 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/quic/core/crypto/quic_crypto_client_config.cc ('k') | net/quic/core/crypto/quic_crypto_server_config.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/crypto/quic_crypto_client_config_test.cc
diff --git a/net/quic/core/crypto/quic_crypto_client_config_test.cc b/net/quic/core/crypto/quic_crypto_client_config_test.cc
index 12d7dbc2ac1a50cfaf0c0e4e77cc67def4c07d64..5d15437143fa28074dbfb456205d8c17f9dd3670 100644
--- a/net/quic/core/crypto/quic_crypto_client_config_test.cc
+++ b/net/quic/core/crypto/quic_crypto_client_config_test.cc
@@ -169,7 +169,7 @@ TEST(QuicCryptoClientConfigTest, CachedState_InitializeFrom) {
TEST(QuicCryptoClientConfigTest, InchoateChlo) {
QuicCryptoClientConfig::CachedState state;
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
- scoped_refptr<QuicCryptoNegotiatedParameters> params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> params(
new QuicCryptoNegotiatedParameters);
CryptoHandshakeMessage msg;
QuicServerId server_id("www.google.com", 443, PRIVACY_MODE_DISABLED);
@@ -196,7 +196,7 @@ TEST(QuicCryptoClientConfigTest, PreferAesGcm) {
TEST(QuicCryptoClientConfigTest, InchoateChloSecure) {
QuicCryptoClientConfig::CachedState state;
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
- scoped_refptr<QuicCryptoNegotiatedParameters> params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> params(
new QuicCryptoNegotiatedParameters);
CryptoHandshakeMessage msg;
QuicServerId server_id("www.google.com", 443, PRIVACY_MODE_DISABLED);
@@ -225,7 +225,7 @@ TEST(QuicCryptoClientConfigTest, InchoateChloSecureWithSCIDNoEXPY) {
&details);
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
- scoped_refptr<QuicCryptoNegotiatedParameters> params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> params(
new QuicCryptoNegotiatedParameters);
CryptoHandshakeMessage msg;
QuicServerId server_id("www.google.com", 443, PRIVACY_MODE_DISABLED);
@@ -251,7 +251,7 @@ TEST(QuicCryptoClientConfigTest, InchoateChloSecureWithSCID) {
QuicWallTime::FromUNIXSeconds(0), &details);
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
- scoped_refptr<QuicCryptoNegotiatedParameters> params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> params(
new QuicCryptoNegotiatedParameters);
CryptoHandshakeMessage msg;
QuicServerId server_id("www.google.com", 443, PRIVACY_MODE_DISABLED);
@@ -267,7 +267,7 @@ TEST(QuicCryptoClientConfigTest, InchoateChloSecureWithSCID) {
TEST(QuicCryptoClientConfigTest, FillClientHello) {
QuicCryptoClientConfig::CachedState state;
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
- scoped_refptr<QuicCryptoNegotiatedParameters> params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> params(
new QuicCryptoNegotiatedParameters);
QuicConnectionId kConnectionId = 1234;
string error_details;
@@ -301,7 +301,7 @@ TEST(QuicCryptoClientConfigTest, ProcessServerDowngradeAttack) {
msg.SetVector(kVER, supported_version_tags);
QuicCryptoClientConfig::CachedState cached;
- scoped_refptr<QuicCryptoNegotiatedParameters> out_params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> out_params(
new QuicCryptoNegotiatedParameters);
string error;
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
@@ -465,7 +465,7 @@ TEST(QuicCryptoClientConfigTest, ProcessReject) {
// Now process the rejection.
QuicCryptoClientConfig::CachedState cached;
- scoped_refptr<QuicCryptoNegotiatedParameters> out_params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> out_params(
new QuicCryptoNegotiatedParameters);
string error;
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
@@ -486,7 +486,7 @@ TEST(QuicCryptoClientConfigTest, ProcessRejectWithLongTTL) {
// Now process the rejection.
QuicCryptoClientConfig::CachedState cached;
- scoped_refptr<QuicCryptoNegotiatedParameters> out_params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> out_params(
new QuicCryptoNegotiatedParameters);
string error;
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
@@ -513,7 +513,7 @@ TEST(QuicCryptoClientConfigTest, ProcessStatelessReject) {
// Now process the rejection.
QuicCryptoClientConfig::CachedState cached;
- scoped_refptr<QuicCryptoNegotiatedParameters> out_params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> out_params(
new QuicCryptoNegotiatedParameters);
string error;
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
@@ -534,7 +534,7 @@ TEST(QuicCryptoClientConfigTest, BadlyFormattedStatelessReject) {
// Now process the rejection.
QuicCryptoClientConfig::CachedState cached;
- scoped_refptr<QuicCryptoNegotiatedParameters> out_params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> out_params(
new QuicCryptoNegotiatedParameters);
string error;
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
@@ -560,7 +560,7 @@ TEST(QuicCryptoClientConfigTest, ServerNonceinSHLO) {
QuicCryptoClientConfig config(CryptoTestUtils::ProofVerifierForTesting());
QuicCryptoClientConfig::CachedState cached;
- scoped_refptr<QuicCryptoNegotiatedParameters> out_params(
+ QuicReferenceCountedPointer<QuicCryptoNegotiatedParameters> out_params(
new QuicCryptoNegotiatedParameters);
string error_details;
EXPECT_EQ(QUIC_INVALID_CRYPTO_MESSAGE_PARAMETER,
« no previous file with comments | « net/quic/core/crypto/quic_crypto_client_config.cc ('k') | net/quic/core/crypto/quic_crypto_server_config.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698