Catch and report exceptions in CalledByNative java methods.

components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java

Index: components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java
diff --git a/components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java b/components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java
index aef180500f480ae5bf9770b4305581bec06d176d..f9088360b7cc312ffee729b3ecb1b191057592fb 100644
--- a/components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java
+++ b/components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java
@@ -7,9 +7,11 @@ package org.chromium.cronet_sample_apk;
 import android.test.suitebuilder.annotation.SmallTest;
 
 import org.chromium.base.test.util.Feature;
+import org.chromium.net.HttpUrlRequest;
 import org.chromium.net.HttpUrlRequestFactoryConfig;
 
 import java.io.File;
+import java.util.HashMap;
 
 /**
  * Example test that just starts the cronet sample.
@@ -91,6 +93,26 @@ public class CronetSampleUrlTest extends CronetSampleTestBase {
 
     @SmallTest
     @Feature({"Cronet"})
+    public void testCalledByNativeException() throws Exception {
+        CronetSampleActivity activity = launchCronetSampleWithUrl(URL);
+
+        // Make sure the activity was created as expected.
+        assertNotNull(activity);
+
+        waitForActiveShellToBeDoneLoading();
+
+        HashMap<String, String> headers = new HashMap<String, String>();
+        // Create request with null listener to trigger an exception.
+        HttpUrlRequest request = activity.mChromiumRequestFactory.createRequest(
+                URL, HttpUrlRequest.REQUEST_PRIORITY_MEDIUM, headers, null);
+        request.start();
+        Thread.sleep(5000);

[Charles, 2014/08/13 21:17:09]

This seems flaky.

[mef, 2014/08/13 21:34:40]

Yeah, what's a good way to make it deterministic? 
Add custom HttpUrlRequestListener listener that signals from onRequestComplete?

[mef, 2014/08/15 14:58:19]

Done. Bad Listener also verifies that onRequestComplete is triggered by
cancellation and thrown exception is preserved in the request.

+        assertTrue(request.isCanceled());
+        assertNotNull(request.getException());
+    }
+
+    @SmallTest
+    @Feature({"Cronet"})
     public void testLegacyLoadUrl() throws Exception {
         HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
         config.enableLegacyMode(true);