[cronet] Pass total bytes received to CronetUrlRequest.onSucceeded()

components/cronet/android/java/src/org/chromium/net/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;
 
 import android.util.Log;
 import android.util.Pair;
 
 import org.chromium.base.CalledByNative;
@@ skipping 598 matching lines @@
         mOnReadCompletedTask.mByteBuffer = byteBuffer;
         postTaskToExecutor(mOnReadCompletedTask);
     }
 
     /**
      * Called when request is completed successfully, no callbacks will be
      * called afterwards.
      */
     @SuppressWarnings("unused")
     @CalledByNative
-    private void onSucceeded() {
-        long totalReceivedBytes;
-        synchronized (mUrlRequestAdapterLock) {
-            if (mUrlRequestAdapter == 0) {
-                return;
-            }
-            totalReceivedBytes =
-                    nativeGetTotalReceivedBytes(mUrlRequestAdapter);
-        }
-
+    private void onSucceeded(long totalReceivedBytes) {
         final NativeExtendedResponseInfo extendedResponseInfo =
                 new NativeExtendedResponseInfo(mResponseInfo,
                         totalReceivedBytes);
         Runnable task = new Runnable() {
             public void run() {
                 synchronized (mUrlRequestAdapterLock) {
                     if (isCanceled()) {
                         return;
                     }
                     // Destroy adapter first, so request context could be shut
@@ skipping 70 matching lines @@
     @NativeClassQualifiedName("CronetURLRequestAdapter")
     private native String nativeGetHttpStatusText(long nativePtr);
 
     @NativeClassQualifiedName("CronetURLRequestAdapter")
     private native String nativeGetNegotiatedProtocol(long nativePtr);
 
     @NativeClassQualifiedName("CronetURLRequestAdapter")
     private native boolean nativeGetWasCached(long nativePtr);
 
     @NativeClassQualifiedName("CronetURLRequestAdapter")
     private native long nativeGetTotalReceivedBytes(long nativePtr);

[mmenke, 2015/04/22 19:13:22]

I don't think we need this any more (And the corresponding C++ code, of course)

[pauljensen, 2015/04/24 11:18:16]

Done.

 }