diff options
author | wtc@chromium.org <wtc@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-22 05:47:51 +0000 |
---|---|---|
committer | wtc@chromium.org <wtc@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-22 05:47:51 +0000 |
commit | 7c1c84c887c0223ebe80f511403037a870fd29e9 (patch) | |
tree | db66b4574d8e6e1b796c8c7a3db218c122b34a71 /net | |
parent | 13ad1815202e9fc7dec1a2046699b298a5dbefb5 (diff) | |
download | chromium_src-7c1c84c887c0223ebe80f511403037a870fd29e9.zip chromium_src-7c1c84c887c0223ebe80f511403037a870fd29e9.tar.gz chromium_src-7c1c84c887c0223ebe80f511403037a870fd29e9.tar.bz2 |
Assert the return value of sock_->Write() for both sync and async
completions.
R=rvargas@chromium.org
BUG=none
TEST=the tests in question still pass.
Review URL: http://codereview.chromium.org/7231003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89988 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net')
-rw-r--r-- | net/socket/transport_client_socket_unittest.cc | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/net/socket/transport_client_socket_unittest.cc b/net/socket/transport_client_socket_unittest.cc index bbccd7c..bb0be35 100644 --- a/net/socket/transport_client_socket_unittest.cc +++ b/net/socket/transport_client_socket_unittest.cc @@ -161,10 +161,9 @@ void TransportClientSocketTest::SendClientRequest() { rv = sock_->Write(request_buffer, arraysize(request_text) - 1, &callback); EXPECT_TRUE(rv >= 0 || rv == ERR_IO_PENDING); - if (rv == ERR_IO_PENDING) { + if (rv == ERR_IO_PENDING) rv = callback.WaitForResult(); - EXPECT_EQ(rv, static_cast<int>(arraysize(request_text) - 1)); - } + EXPECT_EQ(rv, static_cast<int>(arraysize(request_text) - 1)); } // TODO(leighton): Add SCTP to this list when it is ready. |