Chromium Code Reviews| Index: components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java |
| diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java |
| index 000d7cd8093d982391e77775b816e3cd9eb0427c..ae94c3deb2ed2e1b3fa166c705d6673f5bb65cea 100644 |
| --- a/components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java |
| +++ b/components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java |
| @@ -91,8 +91,7 @@ public class QuicTest extends CronetTestBase { |
| assertEquals("quic/1+spdy/3", listener.mResponseInfo.getNegotiatedProtocol()); |
| // The total received bytes should be larger than the content length, to account for |
| // headers. |
| - assertTrue( |
| - listener.mExtendedResponseInfo.getTotalReceivedBytes() > expectedContent.length()); |
| + assertTrue(listener.mResponseInfo.getReceivedBytesCount() > expectedContent.length()); |
|
xunjieli
2015/10/05 20:39:02
Hmm interesting. I didn't know they changed QUIC's
mef
2015/10/05 22:09:19
Apparently I was wrong and headers ARE included in
|
| // This test takes a long time, since the update will only be scheduled |
| // after kUpdatePrefsDelayMs in http_server_properties_manager.cc. |
| @@ -128,8 +127,7 @@ public class QuicTest extends CronetTestBase { |
| assertEquals("quic/1+spdy/3", listener2.mResponseInfo.getNegotiatedProtocol()); |
| // The total received bytes should be larger than the content length, to account for |
| // headers. |
| - assertTrue( |
| - listener2.mExtendedResponseInfo.getTotalReceivedBytes() > expectedContent.length()); |
| + assertTrue(listener2.mResponseInfo.getReceivedBytesCount() > expectedContent.length()); |
| } |
| // Returns whether a file contains a particular string. |