Add HttpUrlConnection backed implementation of CronetEngine.

components/cronet/android/test/javatests/src/org/chromium/net/CronetUploadTest.java

Index: components/cronet/android/test/javatests/src/org/chromium/net/CronetUploadTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/CronetUploadTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUploadTest.java
index 65ac9b89102854f72f73f1e87ba5ae1e725bab4e..f2101cc1340d72080c2edd37efd59077d838489e 100644
--- a/components/cronet/android/test/javatests/src/org/chromium/net/CronetUploadTest.java
+++ b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUploadTest.java
@@ -7,6 +7,7 @@ package org.chromium.net;
 import android.test.suitebuilder.annotation.SmallTest;
 
 import org.chromium.base.test.util.Feature;
+import org.chromium.net.CronetTestBase.OnlyRunNativeCronet;
 
 import java.util.Arrays;
 import java.util.List;
@@ -49,8 +50,8 @@ public class CronetUploadTest extends CronetTestBase {
      */
     @SmallTest
     @Feature({"Cronet"})
-    public void testInitTriggersRewindAndInitBeforeRewindCompletes()
-            throws Exception {
+    @OnlyRunNativeCronet

[pauljensen, 2015/12/21 20:27:58]

why are all of our upload tests disabled?  can we add a comment explaining this?
 should we allow adding an annotation to the class?  this might be useful for
QuicTest and SdchTest also.

[Charles, 2015/12/21 23:36:26]

Annotation for the class is a matter of preference, although it's easier to do
per-method, and also encourages people adding new tests to consider the
environment being tested. Also has less chance of losing test coverage by moving
tests between classes.

These are all disabled because they're testing CronetUploadDatastream, which is
a peer for a native object.

[pauljensen, 2015/12/29 16:35:53]

SGTM
SGTM, looks like CronetUrlRequestTest.java tests uploading.

+    public void testInitTriggersRewindAndInitBeforeRewindCompletes() throws Exception {
         // Init completes synchronously and read succeeds.
         assertTrue(mHandler.init());
         mHandler.read();
@@ -98,8 +99,8 @@ public class CronetUploadTest extends CronetTestBase {
      */
     @SmallTest
     @Feature({"Cronet"})
-    public void testInitTriggersRewindAndInitAfterRewindCompletes()
-            throws Exception {
+    @OnlyRunNativeCronet
+    public void testInitTriggersRewindAndInitAfterRewindCompletes() throws Exception {
         // Init completes synchronously and read succeeds.
         assertTrue(mHandler.init());
         mHandler.read();
@@ -146,6 +147,7 @@ public class CronetUploadTest extends CronetTestBase {
      */
     @SmallTest
     @Feature({"Cronet"})
+    @OnlyRunNativeCronet
     public void testReadCompleteTriggerRewind() throws Exception {
         // Reset and init before read completes.
         assertTrue(mHandler.init());
@@ -180,6 +182,7 @@ public class CronetUploadTest extends CronetTestBase {
      */
     @SmallTest
     @Feature({"Cronet"})
+    @OnlyRunNativeCronet
     public void testReadCompleteTriggerRewindOnlyOneRewind() throws Exception {
         testReadCompleteTriggerRewind();
         // Reset and Init again, no rewind should happen.
@@ -197,8 +200,8 @@ public class CronetUploadTest extends CronetTestBase {
      */
     @SmallTest
     @Feature({"Cronet"})
-    public void testResetBeforeReadCompleteAndInitTriggerRewind()
-            throws Exception {
+    @OnlyRunNativeCronet
+    public void testResetBeforeReadCompleteAndInitTriggerRewind() throws Exception {
         // Reset before read completes. Rewind is not triggered.
         assertTrue(mHandler.init());
         mHandler.read();
@@ -232,8 +235,8 @@ public class CronetUploadTest extends CronetTestBase {
      */
     @SmallTest
     @Feature({"Cronet"})
-    public void testDestroyNativeStreamBeforeReadComplete()
-            throws Exception {
+    @OnlyRunNativeCronet
+    public void testDestroyNativeStreamBeforeReadComplete() throws Exception {
         // Start a read and wait for it to be pending.
         assertTrue(mHandler.init());
         mHandler.read();
@@ -263,8 +266,8 @@ public class CronetUploadTest extends CronetTestBase {
      */
     @SmallTest
     @Feature({"Cronet"})
-    public void testDestroyNativeStreamBeforeRewindComplete()
-            throws Exception {
+    @OnlyRunNativeCronet
+    public void testDestroyNativeStreamBeforeRewindComplete() throws Exception {
         // Start a read and wait for it to complete.
         assertTrue(mHandler.init());
         mHandler.read();