[Cronet] Do not call into native adapter after it is destroyed in ChromiumUrlRequest.java

components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequest.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 org.apache.http.conn.ConnectTimeoutException;
 import org.chromium.base.CalledByNative;
@@ skipping 419 matching lines @@
         ResponseHeadersMap result = new ResponseHeadersMap();
         nativeGetAllHeaders(mUrlRequestAdapter, result);
         return result;
     }
 
     @Override
     public String getUrl() {
         return mUrl;
     }
 
+    /**
+     * Resets native request adapter object. This should only be used in tests.
+     */
+    public void resetAdapterForTesting() {

[mmenke, 2015/02/20 23:19:43]

Rather than have this method, can we run tests on a ChromiumUrlRequest that has
completed - maybe both on one that succeeded and one that failed?

[xunjieli, 2015/02/25 21:31:28]

Done.

+        mUrlRequestAdapter = 0;
+    }
+
     private static int convertRequestPriority(int priority) {
         switch (priority) {
             case HttpUrlRequest.REQUEST_PRIORITY_IDLE:
                 return ChromiumUrlRequestPriority.IDLE;
             case HttpUrlRequest.REQUEST_PRIORITY_LOWEST:
                 return ChromiumUrlRequestPriority.LOWEST;
             case HttpUrlRequest.REQUEST_PRIORITY_LOW:
                 return ChromiumUrlRequestPriority.LOW;
             case HttpUrlRequest.REQUEST_PRIORITY_MEDIUM:
                 return ChromiumUrlRequestPriority.MEDIUM;
@@ skipping 270 matching lines @@
     private native void nativeGetAllHeaders(long urlRequestAdapter,
             ResponseHeadersMap headers);
 
     private native String nativeGetNegotiatedProtocol(long urlRequestAdapter);
 
     // Explicit class to work around JNI-generator generics confusion.
     private static class ResponseHeadersMap extends
             HashMap<String, List<String>> {
     }
 }