Setup initial mock url request job tests for Cronet

components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockURLRequestJobTest.java

Index: components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockURLRequestJobTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockURLRequestJobTest.java b/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockURLRequestJobTest.java
new file mode 100644
index 0000000000000000000000000000000000000000..a602ffb551191df92a06349630293edb77cc8d22
--- /dev/null
+++ b/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockURLRequestJobTest.java
@@ -0,0 +1,100 @@
+// 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.cronet_test_apk;
+
+import android.os.ConditionVariable;
+import android.test.suitebuilder.annotation.SmallTest;
+import android.util.Log;
+
+import org.chromium.base.test.util.Feature;
+import org.chromium.net.HttpUrlRequest;
+import org.chromium.net.HttpUrlRequestListener;
+
+import java.util.HashMap;
+
+// Tests that use mock URL request jobs to simulate URL requests.

[mmenke, 2014/09/18 18:31:45]

nit:  mock URLRequestJobs

[xunjieli, 2014/09/18 22:45:15]

Done.

+public class MockURLRequestJobTest extends CronetTestBase {
+    private static final String TAG = "MockURLRequestJobTest";
+    private static final String MOCK_CRONET_TEST_URL =
+            "http://mock.cronet.success/";
+    private static final String MOCK_CRONET_TEST_FAILED_URL =
+            "http://mock.cronet.failed/";
+
+    class MockHttpUrlRequestListener implements HttpUrlRequestListener {
+        ConditionVariable mComplete = new ConditionVariable();
+        public int httpStatusCode = 0;
+        public String url;

[mmenke, 2014/09/18 18:31:45]

nit:  mHttpStatusCode / mUrl

[xunjieli, 2014/09/18 22:45:15]

Done.

+
+        public MockHttpUrlRequestListener() {
+        }
+
+        @Override
+        public void onResponseStarted(HttpUrlRequest request) {
+            Log.i(TAG, "****** Response Started, content length is "
+                    + request.getContentLength());

[mmenke, 2014/09/18 18:31:45]

nit:  Binary operators on previous line is the preferred style.

[xunjieli, 2014/09/18 22:45:15]

Done. Sry! you mentioned this last time.

+            Log.i(TAG, "*** Headers Are *** " + request.getAllHeaders());
+            httpStatusCode = request.getHttpStatusCode();
+        }
+
+        public void blockForComplete() {
+            mComplete.block();
+        }
+
+        @Override
+        public void onRequestComplete(HttpUrlRequest request) {
+            url = request.getUrl();
+            mComplete.open();
+            Log.i(TAG, "****** Request Complete, status code is "
+                    + request.getHttpStatusCode());

[mmenke, 2014/09/18 18:31:45]

nit:  Binary operators on previous line is the preferred style.

(You may see Cronet code violating this - we basically started with something
that wasn't fully reviewed, and haven't fully cleaned it up).

[xunjieli, 2014/09/18 22:45:15]

Done.

+        }
+    }
+
+    @SmallTest
+    @Feature({"Cronet"})
+    public void testCreateURLRequest() throws Exception {
+        CronetTestActivity activity = launchCronetTestApp();
+        // Make sure the activity was created as expected.
+        assertNotNull(activity);
+
+        HashMap<String, String> headers = new HashMap<String, String>();
+        MockHttpUrlRequestListener listener = new MockHttpUrlRequestListener();
+
+        HttpUrlRequest request = activity.mChromiumRequestFactory.createRequest(
+                MOCK_CRONET_TEST_URL,
+                HttpUrlRequest.REQUEST_PRIORITY_MEDIUM,
+                headers,
+                listener);
+        request.start();
+        listener.blockForComplete();
+
+        // Make sure that the URL is set as expected.
+        assertEquals(MOCK_CRONET_TEST_URL, listener.url);
+        assertEquals(200, listener.httpStatusCode);
+    }
+
+    @SmallTest
+    @Feature({"Cronet"})
+    public void testFailedURLRequest() throws Exception {
+        CronetTestActivity activity = launchCronetTestApp();
+        // Make sure the activity was created as expected.
+        assertNotNull(activity);
+
+        HashMap<String, String> headers = new HashMap<String, String>();
+        MockHttpUrlRequestListener listener = new MockHttpUrlRequestListener();
+
+        HttpUrlRequest request = activity.mChromiumRequestFactory.createRequest(
+                MOCK_CRONET_TEST_FAILED_URL,
+                HttpUrlRequest.REQUEST_PRIORITY_MEDIUM,
+                headers,
+                listener);
+        request.start();
+        listener.blockForComplete();
+
+        // Make sure that the URL is set as expected.
+        assertEquals(MOCK_CRONET_TEST_FAILED_URL, listener.url);
+        assertEquals(0, listener.httpStatusCode);
+    }
+

[mmenke, 2014/09/18 18:31:45]

nit:  Remove blank line.

[xunjieli, 2014/09/18 22:45:15]

Done.

+}