joy/android_packages_modules_Connectivity
Watch
1
Fork
You've already forked android_packages_modules_Connectivity
 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge "Fix thread leak in HttpEngineTest" into main

Browse source
This commit is contained in:
Junyu Lai 2024-01-25 05:58:53 +00:00 • committed by Gerrit Code Review
commit c1f1b29ca4
1 changed files with 2 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Cronet/tests/cts/src/android/net/http/cts/HttpEngineTest.java
View file

@ -247,10 +247,8 @@ public class HttpEngineTest {
@Test
public void testHttpEngine_requestUsesDefaultUserAgent() throws Exception {
mEngine = mEngineBuilder.build();
HttpCtsTestServer server =
new HttpCtsTestServer(ApplicationProvider.getApplicationContext());
String url = server.getUserAgentUrl();
String url = mTestServer.getUserAgentUrl();
UrlRequest request =
mEngine.newUrlRequestBuilder(url, mCallback.getExecutor(), mCallback).build();
request.start();
@ -266,14 +264,12 @@ public class HttpEngineTest {
@Test
public void testHttpEngine_requestUsesCustomUserAgent() throws Exception {
String userAgent = "CtsTests User Agent";
HttpCtsTestServer server =
new HttpCtsTestServer(ApplicationProvider.getApplicationContext());
mEngine =
new HttpEngine.Builder(ApplicationProvider.getApplicationContext())
.setUserAgent(userAgent)
.build();
String url = server.getUserAgentUrl();
String url = mTestServer.getUserAgentUrl();
UrlRequest request =
mEngine.newUrlRequestBuilder(url, mCallback.getExecutor(), mCallback).build();
request.start();