Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(135)

Unified Diff: components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java

Issue 1359343005: Update ResponseInfo to UrlResponseInfo with API review comments. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update README.md Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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());
// 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.

Powered by Google App Engine
This is Rietveld 408576698