| 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 3dea66d9085208e29436dffd68f3b8e9091f3479..52afc3744099eafb412170514194f809a0133ce1 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
|
| @@ -49,7 +49,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);
|
| private final boolean mAllowDirectExecutor;
|
|
|
| /* Native adapter object, owned by UrlRequest. */
|
| @@ -700,10 +700,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 {
|
| @@ -715,7 +716,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));
|
| }
|
|
|