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

Unified Diff: components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/HttpUrlRequestFactoryTest.java

Issue 544223003: Add SetSupportsQuic method to explicitly specify server that supports QUIC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added QUIC hints to HttpUrlRequestFactoryConfig. Created 6 years, 3 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/sample/javatests/src/org/chromium/cronet_sample_apk/HttpUrlRequestFactoryTest.java
diff --git a/components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/HttpUrlRequestFactoryTest.java b/components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/HttpUrlRequestFactoryTest.java
index e37b2d23a014af00a3126a12e305fa09fe1a298a..11455d8e99703b567c61f34037a1dea13a9d8df4 100644
--- a/components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/HttpUrlRequestFactoryTest.java
+++ b/components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/HttpUrlRequestFactoryTest.java
@@ -21,6 +21,10 @@ public class HttpUrlRequestFactoryTest extends InstrumentationTestCase {
@Feature({"Cronet"})
public void testCreateFactory() {
HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
+ config.enableQUIC(true);
+ config.addQuicHint("google.com", 0);
mmenke 2014/09/09 14:53:35 0 uses the default port? Should update addQuicHin
mef 2014/09/10 16:48:18 It was my feeble attempt at guessing correct param
+ config.addQuicHint("youtube.com", 443);
+
HttpUrlRequestFactory factory = HttpUrlRequestFactory.createFactory(
getInstrumentation().getContext(), config);
assertNotNull("Factory should be created", factory);

Powered by Google App Engine
This is Rietveld 408576698