From dc2cc746109141fd48e9df1018204fdf772ac317 Mon Sep 17 00:00:00 2001 From: "rch@chromium.org" Date: Sun, 21 Oct 2012 13:56:13 +0000 Subject: Change CryptoFramer to return a QuicData* instead of returning a bool and setting an out-param to the new QuicData*. This simplifies the API for callers. Merge internal change 35500299 Review URL: https://chromiumcodereview.appspot.com/11184057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163203 0039d316-1c4b-4281-b951-d872f2087c98 --- net/quic/crypto/crypto_framer_test.cc | 22 ++++++++-------------- 1 file changed, 8 insertions(+), 14 deletions(-) (limited to 'net/quic/crypto/crypto_framer_test.cc') diff --git a/net/quic/crypto/crypto_framer_test.cc b/net/quic/crypto/crypto_framer_test.cc index 5d79eed..eed175f 100644 --- a/net/quic/crypto/crypto_framer_test.cc +++ b/net/quic/crypto/crypto_framer_test.cc @@ -100,10 +100,8 @@ TEST(CryptoFramerTest, ConstructHandshakeMessage) { }; CryptoFramer framer; - QuicData* data; - EXPECT_TRUE(framer.ConstructHandshakeMessage(message, &data)); - scoped_ptr scoped_data(data); - + scoped_ptr data(framer.ConstructHandshakeMessage(message)); + ASSERT_TRUE(data.get() != NULL); test::CompareCharArraysWithHexError("constructed packet", data->data(), data->length(), AsChars(packet), arraysize(packet)); @@ -137,9 +135,8 @@ TEST(CryptoFramerTest, ConstructHandshakeMessageWithTwoKeys) { }; CryptoFramer framer; - QuicData* data; - EXPECT_TRUE(framer.ConstructHandshakeMessage(message, &data)); - scoped_ptr scoped_data(data); + scoped_ptr data(framer.ConstructHandshakeMessage(message)); + ASSERT_TRUE(data.get() != NULL); test::CompareCharArraysWithHexError("constructed packet", data->data(), data->length(), @@ -154,10 +151,8 @@ TEST(CryptoFramerTest, ConstructHandshakeMessageTooManyEntries) { } CryptoFramer framer; - - QuicData* dummy = NULL; - EXPECT_FALSE(framer.ConstructHandshakeMessage(message, &dummy)); - scoped_ptr scoped_data(dummy); + scoped_ptr data(framer.ConstructHandshakeMessage(message)); + EXPECT_TRUE(data.get() == NULL); } @@ -167,9 +162,8 @@ TEST(CryptoFramerTest, ConstructHandshakeMessageInvalidLength) { message.tag_value_map[0x12345678] = ""; CryptoFramer framer; - QuicData* dummy = NULL; - EXPECT_FALSE(framer.ConstructHandshakeMessage(message, &dummy)); - scoped_ptr scoped_data(dummy); + scoped_ptr data(framer.ConstructHandshakeMessage(message)); + EXPECT_TRUE(data.get() == NULL); } TEST(CryptoFramerTest, ProcessInput) { -- cgit v1.1