Index: components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java |
diff --git a/components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java b/components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java |
index d7da13995e5e93778193615b2acd737971a373fa..6fd0a479e8299aaed82d7440af74301a603744b4 100644 |
--- a/components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java |
+++ b/components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java |
@@ -48,7 +48,7 @@ import javax.annotation.concurrent.GuardedBy; |
@VisibleForTesting |
public final class CronetUrlRequest implements UrlRequest { |
private static final RequestFinishedInfo.Metrics EMPTY_METRICS = |
- new RequestFinishedInfo.Metrics(null, null, null, null); |
+ new CronetMetrics(null, null, null, null); |
/* Native adapter object, owned by UrlRequest. */ |
@GuardedBy("mUrlRequestAdapterLock") |
@@ -696,10 +696,11 @@ public final class CronetUrlRequest implements UrlRequest { |
} |
RequestFinishedInfo getRequestFinishedInfo() { |
+ // TODO(mgersh): fill in real values for finishedReason and exception |
return new RequestFinishedInfo(mInitialUrl, mRequestAnnotations, |
(mRequestMetricsAccumulator != null ? mRequestMetricsAccumulator.getRequestMetrics() |
: EMPTY_METRICS), |
- mResponseInfo); |
+ RequestFinishedInfo.SUCCEEDED, mResponseInfo, null); |
} |
private final class UrlRequestMetricsAccumulator { |
@@ -711,7 +712,7 @@ public final class CronetUrlRequest implements UrlRequest { |
private Long mTotalTimeMs; |
private RequestFinishedInfo.Metrics getRequestMetrics() { |
- return new RequestFinishedInfo.Metrics(mTtfbMs, mTotalTimeMs, |
+ return new CronetMetrics(mTtfbMs, mTotalTimeMs, |
null, // TODO(klm): Compute sentBytesCount. |
(mResponseInfo != null ? mResponseInfo.getReceivedBytesCount() : 0)); |
} |