Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(136)

Unified Diff: components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlTest.java

Issue 1039693003: [Cronet] Add support to run quic_simple_server (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Suppress fb warning Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlTest.java
index 481cebad8cc2df09f5a9c25b6b8cf60c5010c74d..385dfb3059df7c033425ad9d4e7914572e82e611 100644
--- a/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlTest.java
+++ b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlTest.java
@@ -134,50 +134,6 @@ public class CronetUrlTest extends CronetTestBase {
@SmallTest
@Feature({"Cronet"})
- public void disabled_testQuicLoadUrl() throws Exception {
- 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";
- config.enableQUIC(true);
- config.addQuicHint("www.google.com", 443, 443);
- config.setExperimentalQuicConnectionOptions("PACE,IW10,FOO,DEADBEEF");
-
- String[] commandLineArgs = {
- CronetTestActivity.CONFIG_KEY, config.toString() };
- CronetTestActivity activity =
- launchCronetTestAppWithUrlAndCommandLineArgs(null,
- commandLineArgs);
- 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);
- activity.stopNetLog();
- }
-
- @SmallTest
- @Feature({"Cronet"})
public void testLegacyLoadUrl() throws Exception {
HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
config.enableLegacyMode(true);

Powered by Google App Engine
This is Rietveld 408576698