diff options
| author | rch <rch@chromium.org> | 2015-05-05 13:02:25 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-05-05 20:03:35 +0000 |
| commit | fbeeeba5c66301c0df903e86c23d6062e4347e82 (patch) | |
| tree | a63c7045aeec07b51ab17b4f1c92d085ddeed316 /net | |
| parent | f3abeea37887a3e4fa22060c3683304bcc93c06b (diff) | |
| download | chromium_src-fbeeeba5c66301c0df903e86c23d6062e4347e82.zip chromium_src-fbeeeba5c66301c0df903e86c23d6062e4347e82.tar.gz chromium_src-fbeeeba5c66301c0df903e86c23d6062e4347e82.tar.bz2 | |
Remove bogus expectations from SpdyNetworkTransactionTest.SynReplyHeadersVary
These caused problems when I attempted to convert this test to use
SequencedSocketData which does not introduce artificial delays.
http://build.chromium.org/p/chromium.fyi/builders/CrWinClang%28dbg%29%20tester/builds/257/steps/net_unittests/logs/Spdy_SpdyNetworkTransactionTest.SynReplyHeadersVary_0
Review URL: https://codereview.chromium.org/1124813006
Cr-Commit-Position: refs/heads/master@{#328390}
Diffstat (limited to 'net')
| -rw-r--r-- | net/spdy/spdy_network_transaction_unittest.cc | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/net/spdy/spdy_network_transaction_unittest.cc b/net/spdy/spdy_network_transaction_unittest.cc index 0e5a701..a29ecf4 100644 --- a/net/spdy/spdy_network_transaction_unittest.cc +++ b/net/spdy/spdy_network_transaction_unittest.cc @@ -3326,14 +3326,6 @@ TEST_P(SpdyNetworkTransactionTest, SynReplyHeadersVary) { EXPECT_EQ("hello!", out.response_data) << i; // Test the response information. - EXPECT_TRUE(out.response_info.response_time > - out.response_info.request_time) << i; - base::TimeDelta test_delay = out.response_info.response_time - - out.response_info.request_time; - base::TimeDelta min_expected_delay; - min_expected_delay.FromMilliseconds(10); - EXPECT_GT(test_delay.InMillisecondsF(), - min_expected_delay.InMillisecondsF()) << i; EXPECT_EQ(out.response_info.vary_data.is_valid(), test_cases[i].vary_matches) << i; |