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

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

Issue 1580583002: Add a whitelist for QUIC hosts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix comments Created 4 years, 11 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/QuicTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java
index 13df2eeb496af509918f538c99e95d45982ef90d..4854cb5b166c174fa20649b55871f2b1819d0ae9 100644
--- a/components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java
+++ b/components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java
@@ -42,6 +42,7 @@ public class QuicTest extends CronetTestBase {
JSONObject quicParams = new JSONObject()
.put("connection_options", "PACE,IW10,FOO,DEADBEEF")
+ .put("host_whitelist", "test.example.com")
.put("store_server_configs_in_properties", true)
.put("delay_tcp_race", true)
.put("max_number_of_lossy_connections", 10)
@@ -115,7 +116,6 @@ public class QuicTest extends CronetTestBase {
// The total received bytes should be larger than the content length, to account for
// headers.
assertTrue(callback.mResponseInfo.getReceivedBytesCount() > expectedContent.length());
-
// This test takes a long time, since the update will only be scheduled
// after kUpdatePrefsDelayMs in http_server_properties_manager.cc.
while (true) {
@@ -138,6 +138,9 @@ public class QuicTest extends CronetTestBase {
builder.setStoragePath(CronetTestFramework.getTestStorage(getContext()));
builder.enableHttpCache(CronetEngine.Builder.HTTP_CACHE_DISK, 1000 * 1024);
builder.enableQUIC(true);
+ JSONObject quicParams = new JSONObject().put("host_whitelist", "test.example.com");
+ JSONObject experimentalOptions = new JSONObject().put("QUIC", quicParams);
+ builder.setExperimentalOptions(experimentalOptions.toString());
builder.setMockCertVerifierForTesting(MockCertVerifier.createMockCertVerifier(CERTS_USED));
mTestFramework = startCronetTestFrameworkWithUrlAndCronetEngineBuilder(null, builder);
registerHostResolver(mTestFramework);

Powered by Google App Engine
This is Rietveld 408576698