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

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

Issue 1817553002: Add host resolver rules experimental flag for Cronet (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: delete registerHostResolverProc and rebase Created 4 years, 5 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/SdchTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/SdchTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/SdchTest.java
index 379403c72eb8602605d7ddb2c34d1f07842d0bcf..07f8e755f4beccfeefffb0e777270b3c4c4e9f00 100644
--- a/components/cronet/android/test/javatests/src/org/chromium/net/SdchTest.java
+++ b/components/cronet/android/test/javatests/src/org/chromium/net/SdchTest.java
@@ -9,6 +9,8 @@ import android.test.suitebuilder.annotation.SmallTest;
import org.chromium.base.test.util.Feature;
import org.chromium.net.CronetTestBase.OnlyRunNativeCronet;
+import org.json.JSONException;
+import org.json.JSONObject;
import java.io.BufferedReader;
import java.io.FileReader;
@@ -36,7 +38,7 @@ public class SdchTest extends CronetTestBase {
}
@SuppressWarnings("deprecation")
- private void setUp(Sdch setting, Api api) {
+ private void setUp(Sdch setting, Api api) throws JSONException {
List<String> commandLineArgs = new ArrayList<String>();
commandLineArgs.add(CronetTestFramework.CACHE_KEY);
commandLineArgs.add(CronetTestFramework.CACHE_DISK);
@@ -54,9 +56,13 @@ public class SdchTest extends CronetTestBase {
}
String[] args = new String[commandLineArgs.size()];
- mTestFramework = startCronetTestFrameworkWithUrlAndCommandLineArgs(
- null, commandLineArgs.toArray(args));
- registerHostResolver(mTestFramework, api == Api.LEGACY);
+ CronetEngine.Builder builder = new CronetEngine.Builder(getContext());
+ JSONObject hostResolverParams = CronetTestUtil.generateHostResolverRules();
+ JSONObject experimentalOptions =
+ new JSONObject().put("HostResolverRules", hostResolverParams);
+ builder.setExperimentalOptions(experimentalOptions.toString());
+ mTestFramework =
+ new CronetTestFramework(null, commandLineArgs.toArray(args), getContext(), builder);
// Start NativeTestServer.
assertTrue(NativeTestServer.startNativeTestServer(getContext()));
}
@@ -163,10 +169,6 @@ public class SdchTest extends CronetTestBase {
assertEquals(200, callback2.mResponseInfo.getHttpStatusCode());
assertEquals("The quick brown fox jumps over the lazy dog.\n", callback2.mResponseAsString);
- // Wait for a bit until SimpleCache finished closing entries before
- // calling shutdown on the CronetEngine.
- // TODO(xunjieli): Remove once crbug.com/486120 is fixed.
- Thread.sleep(5000);
mTestFramework.mCronetEngine.shutdown();
// Shutting down the context will make JsonPrefStore to flush pending
@@ -179,7 +181,6 @@ public class SdchTest extends CronetTestBase {
null, mTestFramework.getCronetEngineBuilder());
CronetUrlRequestContext newContext = (CronetUrlRequestContext) mTestFramework.mCronetEngine;
long newContextAdapter = getContextAdapter(newContext);
- registerHostResolver(mTestFramework);
DictionaryAddedObserver newObserver =
new DictionaryAddedObserver(targetUrl, newContextAdapter, false /** Legacy Api */);
newObserver.waitForDictionaryAdded();

Powered by Google App Engine
This is Rietveld 408576698