Commit f55a1bb6 authored by Caleb Raitto's avatar Caleb Raitto Committed by Commit Bot

[Cronet] Rename reportFinished() to reportRequestFinished().

This is the Java-side followup to the rename request in the native API
made here [1].

[1] https://crrev.com/c/1496296/26/components/cronet/native/engine.h#71

Bug: 879208
Change-Id: I0eb0bcc8c7e7f1a2376c271e45f56b9f457c22f2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574227Reviewed-by: default avatarPaul Jensen <pauljensen@chromium.org>
Reviewed-by: default avatarMisha Efimov <mef@chromium.org>
Commit-Queue: Caleb Raitto <caraitto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652548}
parent 9bdce258
...@@ -679,7 +679,7 @@ public class CronetBidirectionalStream extends ExperimentalBidirectionalStream { ...@@ -679,7 +679,7 @@ public class CronetBidirectionalStream extends ExperimentalBidirectionalStream {
} }
final RequestFinishedInfo requestFinishedInfo = new RequestFinishedInfoImpl(mInitialUrl, final RequestFinishedInfo requestFinishedInfo = new RequestFinishedInfoImpl(mInitialUrl,
mRequestAnnotations, mMetrics, finishedReason, mResponseInfo, mException); mRequestAnnotations, mMetrics, finishedReason, mResponseInfo, mException);
mRequestContext.reportFinished(requestFinishedInfo); mRequestContext.reportRequestFinished(requestFinishedInfo);
} }
} }
......
...@@ -802,7 +802,7 @@ public final class CronetUrlRequest extends UrlRequestBase { ...@@ -802,7 +802,7 @@ public final class CronetUrlRequest extends UrlRequestBase {
if (mMetrics != null) { if (mMetrics != null) {
final RequestFinishedInfo requestInfo = new RequestFinishedInfoImpl(mInitialUrl, final RequestFinishedInfo requestInfo = new RequestFinishedInfoImpl(mInitialUrl,
mRequestAnnotations, mMetrics, mFinishedReason, mResponseInfo, mException); mRequestAnnotations, mMetrics, mFinishedReason, mResponseInfo, mException);
mRequestContext.reportFinished(requestInfo); mRequestContext.reportRequestFinished(requestInfo);
if (mRequestFinishedListener != null) { if (mRequestFinishedListener != null) {
try { try {
mRequestFinishedListener.getExecutor().execute(new Runnable() { mRequestFinishedListener.getExecutor().execute(new Runnable() {
......
...@@ -644,7 +644,7 @@ public class CronetUrlRequestContext extends CronetEngineBase { ...@@ -644,7 +644,7 @@ public class CronetUrlRequestContext extends CronetEngineBase {
} }
} }
void reportFinished(final RequestFinishedInfo requestInfo) { void reportRequestFinished(final RequestFinishedInfo requestInfo) {
ArrayList<VersionSafeCallbacks.RequestFinishedInfoListener> currentListeners; ArrayList<VersionSafeCallbacks.RequestFinishedInfoListener> currentListeners;
synchronized (mFinishedListenerLock) { synchronized (mFinishedListenerLock) {
if (mFinishedListenerMap.isEmpty()) return; if (mFinishedListenerMap.isEmpty()) return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment