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

Unified Diff: components/cronet/android/api/src/org/chromium/net/JavaCronetEngine.java

Issue 2417643007: Expose RTT and throughput estimates from Cronet (Closed)
Patch Set: Removed annotation Created 4 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/api/src/org/chromium/net/JavaCronetEngine.java
diff --git a/components/cronet/android/api/src/org/chromium/net/JavaCronetEngine.java b/components/cronet/android/api/src/org/chromium/net/JavaCronetEngine.java
index ab19536e7fc18124d751af521fc86d31f468d7fb..71b2015b6bf817d86df499dc9035d447dbfcb1a8 100644
--- a/components/cronet/android/api/src/org/chromium/net/JavaCronetEngine.java
+++ b/components/cronet/android/api/src/org/chromium/net/JavaCronetEngine.java
@@ -113,6 +113,21 @@ final class JavaCronetEngine extends CronetEngine {
}
@Override
+ public int getHttpRttMs() {
+ return RttThroughputValues.INVALID_RTT_THROUGHPUT;
+ }
+
+ @Override
+ public int getTransportRttMs() {
+ return RttThroughputValues.INVALID_RTT_THROUGHPUT;
+ }
+
+ @Override
+ public int getDownstreamThroughputKbps() {
+ return RttThroughputValues.INVALID_RTT_THROUGHPUT;
+ }
+
+ @Override
public void configureNetworkQualityEstimatorForTesting(
boolean useLocalHostRequests, boolean useSmallerResponses) {}

Powered by Google App Engine
This is Rietveld 408576698