Change RequestFinishedListener to provide executor

components/cronet/android/java/src/org/chromium/net/impl/CronetUrlRequest.java

 // Copyright 2014 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.impl;
 
 import android.os.SystemClock;
 import android.support.annotation.Nullable;
 import android.util.Log;
 
@@ skipping 474 matching lines @@
         };
         postTaskToExecutor(task);
     }
 
     ////////////////////////////////////////////////
     // Private methods called by the native code.
     // Always called on network thread.
     ////////////////////////////////////////////////
 
     /**
-     * Called before following redirects. The redirect will automatically be
-     * followed, unless the request is paused or canceled during this
-     * callback. If the redirect response has a body, it will be ignored.
+     * Called before following redirects. The redirect will only be followed if
+     * {@link #followRedirect()} is called. If the redirect response has a body, it will be ignored.
      * This will only be called between start and onResponseStarted.
      *
      * @param newLocation Location where request is redirected.
      * @param httpStatusCode from redirect response
      * @param receivedBytesCount count of bytes received for redirect response
-     * @param headers an array of response headers with keys at the even indices
-     *         followed by the corresponding values at the odd indices.
+     * @param headers an array of response headers with keys at the even indices followed by the

[xunjieli, 2016/07/29 16:35:24]

Why is this line changed? Could you revert?

[mgersh, 2016/07/29 18:21:52]

Done.

+     * corresponding values at the odd indices.
      */
     @SuppressWarnings("unused")
     @CalledByNative
     private void onRedirectReceived(final String newLocation, int httpStatusCode,
             String httpStatusText, String[] headers, boolean wasCached, String negotiatedProtocol,
             String proxyServer, long receivedBytesCount) {
         final UrlResponseInfo responseInfo = prepareResponseInfoOnNetworkThread(httpStatusCode,
                 httpStatusText, headers, wasCached, negotiatedProtocol, proxyServer);
         mReceivedBytesCountFromRedirects += receivedBytesCount;
         responseInfo.setReceivedBytesCount(mReceivedBytesCountFromRedirects);
@@ skipping 243 matching lines @@
     @NativeClassQualifiedName("CronetURLRequestAdapter")
     private native boolean nativeReadData(
             long nativePtr, ByteBuffer byteBuffer, int position, int capacity);
 
     @NativeClassQualifiedName("CronetURLRequestAdapter")
     private native void nativeDestroy(long nativePtr, boolean sendOnCanceled);
 
     @NativeClassQualifiedName("CronetURLRequestAdapter")
     private native void nativeGetStatus(long nativePtr, UrlRequest.StatusListener listener);
 }