[Cronet] Add support to run quic_simple_server

components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlTest.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.content.Context;
 import android.test.suitebuilder.annotation.SmallTest;
 
 import org.chromium.base.PathUtils;
@@ skipping 116 matching lines @@
         try {
             request.setUploadData(null, uploadData);
             fail("setUploadData should throw on null content type");
         } catch (NullPointerException e) {
             // Nothing to do here.
         }
     }
 
     @SmallTest
     @Feature({"Cronet"})
-    public void disabled_testQuicLoadUrl() throws Exception {
+    public void testQuicLoadUrl() throws Exception {
+        // Loads library first since native functions are used to retrieve
+        // QuicTestServer info before config is constructed.
+        System.loadLibrary("cronet_tests");

[mef, 2015/04/06 20:30:56]

Thanks for the comment! 
That's annoying that we have to do this, but I don't have better ideas.

Would it make sense to extract QUIC tests into separate test class, and load
library in its SetUp()?

[xunjieli, 2015/04/06 21:23:50]

Done. Makes sense, since we will be adding more tests later on.

         HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
-        // TODO(mef): Test Quic end-to-end using local QUIC server.
-        String quicURL = "https://www.google.com:443";
-        String quicNegotiatedProtocol = "quic/1+spdy/3";
+        String quicURL = QuicTestServer.getServerURL() + "/simple.txt";
         config.enableQUIC(true);
-        config.addQuicHint("www.google.com", 443, 443);
+        config.setLibraryName("cronet_tests");
+        config.addQuicHint(QuicTestServer.getServerHost(),
+                QuicTestServer.getServerPort(), QuicTestServer.getServerPort());
         config.setExperimentalQuicConnectionOptions("PACE,IW10,FOO,DEADBEEF");
 
         String[] commandLineArgs = {
                 CronetTestActivity.CONFIG_KEY, config.toString() };
         CronetTestActivity activity =
                 launchCronetTestAppWithUrlAndCommandLineArgs(null,
                                                              commandLineArgs);
+        QuicTestServer.startQuicTestServer(activity.getApplicationContext());
         activity.startNetLog();
 
         HashMap<String, String> headers = new HashMap<String, String>();
         TestHttpUrlRequestListener listener = new TestHttpUrlRequestListener();
 
-        // Try several times as first request may not use QUIC.
-        // TODO(mef): Remove loop after adding http server properties manager.
-        for (int i = 0; i < 10; ++i) {
-            HttpUrlRequest request =
-                    activity.mRequestFactory.createRequest(
-                            quicURL,
-                            HttpUrlRequest.REQUEST_PRIORITY_MEDIUM,
-                            headers,
-                            listener);
-            request.start();
-            listener.blockForComplete();
-            assertEquals(200, listener.mHttpStatusCode);
-            if (listener.mNegotiatedProtocol.equals(quicNegotiatedProtocol)) {
-                break;
-            }
-            Thread.sleep(1000);
-            listener.resetComplete();
-        }
-
-        assertEquals(quicNegotiatedProtocol, listener.mNegotiatedProtocol);
+        HttpUrlRequest request =
+                activity.mRequestFactory.createRequest(
+                        quicURL,
+                        HttpUrlRequest.REQUEST_PRIORITY_MEDIUM,
+                        headers,
+                        listener);
+        request.start();
+        listener.blockForComplete();
+        assertEquals(200, listener.mHttpStatusCode);
+        assertEquals(
+                "This is a simple text file served by QUIC.\n", listener.mResponseAsString);
+        assertEquals("quic/1+spdy/3", listener.mNegotiatedProtocol);
         activity.stopNetLog();
+        QuicTestServer.shutdownQuicTestServer();
     }
 
     @SmallTest
     @Feature({"Cronet"})
     public void testLegacyLoadUrl() throws Exception {
         HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
         config.enableLegacyMode(true);
         // TODO(mef) fix tests so that library isn't loaded for legacy stack
         config.setLibraryName("cronet_tests");
 
@@ skipping 21 matching lines @@
                 URL, HttpUrlRequest.REQUEST_PRIORITY_MEDIUM, headers, listener);
         request.setHttpMethod("HEAD");
         request.start();
         listener.blockForComplete();
         assertEquals(200, listener.mHttpStatusCode);
         // HEAD requests do not get any response data and Content-Length must be
         // ignored.
         assertEquals(0, listener.mResponseAsBytes.length);
     }
 }