Change RequestFinishedListener to provide executor

components/cronet/android/api/src/org/chromium/net/CronetEngine.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.net;
 
 import android.annotation.SuppressLint;
 import android.content.Context;
 import android.net.http.HttpResponseCache;
 import android.support.annotation.IntDef;
@@ skipping 820 matching lines @@
      * be called on {@code executor}'s thread. {@code executor} must not run
      * tasks on the current thread to prevent blocking networking operations
      * and causing exceptions during shutdown.
      *
      * @param url {@link URL} for the request.
      * @param callback callback object that gets invoked on different events.
      * @param executor {@link Executor} on which all callbacks will be invoked.
      * @param priority priority of the request which should be one of the
      *         {@link UrlRequest.Builder#REQUEST_PRIORITY_IDLE REQUEST_PRIORITY_*}
      *         values.
-     * @param requestAnnotations Objects to pass on to {@link CronetEngine.RequestFinishedListener}.
+     * @param requestAnnotations Objects to pass on to {@link RequestFinishedListener}.
      * @param disableCache disables cache for the request.
      *         If context is not set up to use cache this param has no effect.
      * @param disableConnectionMigration disables connection migration for this
      *         request if it is enabled for the session.
      * @return new request.
      * @deprecated Use {@link UrlRequest.Builder#build}.
-     * @hide as it references hidden CronetEngine.RequestFinishedListener
+     * @hide as it references hidden RequestFinishedListener
      */
     @Deprecated
     protected abstract UrlRequest createRequest(String url, UrlRequest.Callback callback,
             Executor executor, int priority, Collection<Object> requestAnnotations,
             boolean disableCache, boolean disableConnectionMigration);
 
     /**
      * Creates a {@link BidirectionalStream} object. {@code callback} methods will
      * be invoked on {@code executor}. {@code executor} must not run
      * tasks on the current thread to prevent blocking networking operations
@@ skipping 402 matching lines @@
         }
 
         /**
          * Returns total bytes received over the network transport layer, or null if not collected.
          */
         @Nullable
         public Long getReceivedBytesCount() {
             return mReceivedBytesCount;
         }
     }
-
-    /**
-     * Interface to listen for finished requests that were created via this CronetEngine instance.
-     *
-     * TODO(tbansal):  http://crbug.com/618034 Remove this API, and replace it with a listener
-     * whose executor is bound to the lifetime of the listener.
-     * @hide as it's a prototype.
-     */
-    public interface RequestFinishedListener {
-        /**
-         * Invoked with request info.
-         * @param requestInfo {@link UrlRequestInfo} for finished request.
-         */
-        void onRequestFinished(UrlRequestInfo requestInfo);
-    }
 }