diff options
| author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-15 17:29:39 +0000 |
|---|---|---|
| committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-15 17:29:39 +0000 |
| commit | e444863cc211ee330361cbf41616005a4ff91a09 (patch) | |
| tree | 412ce9f15bf32c37c1ab135a6fdea4b0e5d6d72d /net/http/http_network_layer_unittest.cc | |
| parent | 9f1087e1370ecdbb3a9670a9ab542e317c00bb7d (diff) | |
| download | chromium_src-e444863cc211ee330361cbf41616005a4ff91a09.zip chromium_src-e444863cc211ee330361cbf41616005a4ff91a09.tar.gz chromium_src-e444863cc211ee330361cbf41616005a4ff91a09.tar.bz2 | |
Fix segfault in HttpNetworkLayerTest.GoogleGET when network connection fails.
This is rather a workaround for the root cause (test touches the network),
but it should not crash anyway.
After this patch the test will fail, not segfault.
TEST=Covered by net_unittest. Check with network connection disabled.
BUG=http://crbug.com/13195
Review URL: http://codereview.chromium.org/125128
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18398 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_network_layer_unittest.cc')
| -rw-r--r-- | net/http/http_network_layer_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/http/http_network_layer_unittest.cc b/net/http/http_network_layer_unittest.cc index 335cb93..46e0f8d 100644 --- a/net/http/http_network_layer_unittest.cc +++ b/net/http/http_network_layer_unittest.cc @@ -69,7 +69,7 @@ TEST_F(HttpNetworkLayerTest, GoogleGET) { int rv = trans->Start(&request_info, &callback); if (rv == net::ERR_IO_PENDING) rv = callback.WaitForResult(); - EXPECT_EQ(net::OK, rv); + ASSERT_EQ(net::OK, rv); std::string contents; rv = ReadTransaction(trans.get(), &contents); |