| Index: components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java
|
| diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java
|
| index 1e433d90b421694647c8879178a37bb272a7adc3..1015855accd078cdf931f38e802d1802113f2554 100644
|
| --- a/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java
|
| +++ b/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java
|
| @@ -12,6 +12,8 @@ import android.os.Looper;
|
| import android.os.StrictMode;
|
| import android.test.suitebuilder.annotation.SmallTest;
|
|
|
| +import org.json.JSONObject;
|
| +
|
| import org.chromium.base.FileUtils;
|
| import org.chromium.base.PathUtils;
|
| import org.chromium.base.annotations.JNINamespace;
|
| @@ -21,7 +23,6 @@ import org.chromium.net.TestUrlRequestCallback.ResponseStep;
|
| import org.chromium.net.impl.CronetLibraryLoader;
|
| import org.chromium.net.impl.CronetUrlRequestContext;
|
| import org.chromium.net.test.EmbeddedTestServer;
|
| -import org.json.JSONObject;
|
|
|
| import java.io.BufferedReader;
|
| import java.io.File;
|
| @@ -160,7 +161,6 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| cronetEngineBuilder.enableLegacyMode(true);
|
| }
|
| cronetEngineBuilder.setUserAgent(userAgentValue);
|
| - cronetEngineBuilder.setLibraryName("cronet_tests");
|
| final CronetTestFramework testFramework =
|
| startCronetTestFrameworkWithUrlAndCronetEngineBuilder(mUrl, cronetEngineBuilder);
|
| NativeTestServer.shutdownNativeTestServer(); // startNativeTestServer returns false if it's
|
| @@ -182,7 +182,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| final CronetTestFramework testFramework = startCronetTestFrameworkAndSkipLibraryInit();
|
|
|
| // Ensure native code is loaded before trying to start test server.
|
| - new CronetEngine.Builder(getContext()).setLibraryName("cronet_tests").build().shutdown();
|
| + new CronetEngine.Builder(getContext()).build().shutdown();
|
|
|
| assertTrue(NativeTestServer.startNativeTestServer(getContext()));
|
| if (!NativeTestServer.isDataReductionProxySupported()) {
|
| @@ -197,7 +197,6 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| cronetEngineBuilder.enableDataReductionProxy("test-key");
|
| cronetEngineBuilder.setDataReductionProxyOptions(serverHostPort, "unused.net:9999",
|
| NativeTestServer.getFileURL("/secureproxychecksuccess.txt"));
|
| - cronetEngineBuilder.setLibraryName("cronet_tests");
|
| testFramework.mCronetEngine = cronetEngineBuilder.build();
|
| TestUrlRequestCallback callback = new TestUrlRequestCallback();
|
|
|
| @@ -521,8 +520,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| Context context = getContext();
|
| File directory = new File(PathUtils.getDataDirectory());
|
| File file = File.createTempFile("cronet", "json", directory);
|
| - CronetEngine cronetEngine = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| + CronetEngine cronetEngine = new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
| // Start NetLog immediately after the request context is created to make
|
| // sure that the call won't crash the app even when the native request
|
| // context is not fully initialized. See crbug.com/470196.
|
| @@ -552,8 +550,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| assertFalse(netLogDir.exists());
|
| assertTrue(netLogDir.mkdir());
|
| File eventFile = new File(netLogDir, "event_file_0.json");
|
| - CronetEngine cronetEngine = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| + CronetEngine cronetEngine = new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
| // Start NetLog immediately after the request context is created to make
|
| // sure that the call won't crash the app even when the native request
|
| // context is not fully initialized. See crbug.com/470196.
|
| @@ -582,8 +579,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| Context context = getContext();
|
| File directory = new File(PathUtils.getDataDirectory());
|
| File file = File.createTempFile("cronet", "json", directory);
|
| - CronetEngine cronetEngine = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| + CronetEngine cronetEngine = new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
| cronetEngine.startNetLogToFile(file.getPath(), false);
|
|
|
| // Start a request.
|
| @@ -613,8 +609,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| assertFalse(netLogDir.exists());
|
| assertTrue(netLogDir.mkdir());
|
| File eventFile = new File(netLogDir, "event_file_0.json");
|
| - CronetEngine cronetEngine = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| + CronetEngine cronetEngine = new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
| cronetEngine.startNetLogToDisk(netLogDir.getPath(), false, MAX_FILE_SIZE);
|
|
|
| // Start a request.
|
| @@ -641,10 +636,8 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| File directory = new File(PathUtils.getDataDirectory());
|
| File file1 = File.createTempFile("cronet1", "json", directory);
|
| File file2 = File.createTempFile("cronet2", "json", directory);
|
| - CronetEngine cronetEngine1 = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| - CronetEngine cronetEngine2 = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| + CronetEngine cronetEngine1 = new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
| + CronetEngine cronetEngine2 = new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
|
|
| cronetEngine1.startNetLogToFile(file1.getPath(), false);
|
| cronetEngine2.startNetLogToFile(file2.getPath(), false);
|
| @@ -690,10 +683,10 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| File eventFile1 = new File(netLogDir1, "event_file_0.json");
|
| File eventFile2 = new File(netLogDir2, "event_file_0.json");
|
|
|
| - CronetUrlRequestContext cronetEngine1 = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| - CronetUrlRequestContext cronetEngine2 = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| + CronetUrlRequestContext cronetEngine1 =
|
| + new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
| + CronetUrlRequestContext cronetEngine2 =
|
| + new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
|
|
| cronetEngine1.startNetLogToDisk(netLogDir1.getPath(), false, MAX_FILE_SIZE);
|
| cronetEngine2.startNetLogToDisk(netLogDir2.getPath(), false, MAX_FILE_SIZE);
|
| @@ -1003,8 +996,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| Context context = getContext();
|
| File directory = new File(PathUtils.getDataDirectory());
|
| File file = File.createTempFile("cronet", "json", directory);
|
| - CronetEngine cronetEngine = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| + CronetEngine cronetEngine = new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
| // Start NetLog with logAll as true.
|
| cronetEngine.startNetLogToFile(file.getPath(), true);
|
| // Start a request.
|
| @@ -1031,8 +1023,8 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| assertFalse(netLogDir.exists());
|
| assertTrue(netLogDir.mkdir());
|
| File eventFile = new File(netLogDir, "event_file_0.json");
|
| - CronetUrlRequestContext cronetEngine = new CronetUrlRequestContext(
|
| - new CronetEngine.Builder(context).setLibraryName("cronet_tests"));
|
| + CronetUrlRequestContext cronetEngine =
|
| + new CronetUrlRequestContext(new CronetEngine.Builder(context));
|
| // Start NetLog with logAll as true.
|
| cronetEngine.startNetLogToDisk(netLogDir.getPath(), true, MAX_FILE_SIZE);
|
| // Start a request.
|
| @@ -1393,7 +1385,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| public void testSkipLibraryLoading() throws Exception {
|
| CronetEngine.Builder builder = new CronetEngine.Builder(getContext());
|
| TestBadLibraryLoader loader = new TestBadLibraryLoader();
|
| - builder.setLibraryLoader(loader).setLibraryName("cronet_tests");
|
| + builder.setLibraryLoader(loader);
|
| try {
|
| // ensureInitialized() calls native code to check the version right after library load
|
| // and will error with the message below if library loading was skipped
|
| @@ -1413,8 +1405,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {
|
| final ConditionVariable uiThreadDone = new ConditionVariable();
|
| new Handler(Looper.getMainLooper()).post(new Runnable() {
|
| public void run() {
|
| - final CronetEngine.Builder builder =
|
| - new CronetEngine.Builder(getContext()).setLibraryName("cronet_tests");
|
| + final CronetEngine.Builder builder = new CronetEngine.Builder(getContext());
|
| new Thread() {
|
| public void run() {
|
| CronetEngine cronetEngine = builder.build();
|
|
|