components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java - Issue 1093793002: Allow setting Cronet NetLog level to LOG_ALL

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

Issue 1093793002: Allow setting Cronet NetLog level to LOG_ALL (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Created 5 years, 8 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

« components/cronet/android/java/src/org/chromium/net/HttpUrlRequestFactory.java ('K') | « components/cronet/android/sample/src/org/chromium/cronet_sample_apk/CronetSampleActivity.java ('k') | components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlTest.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

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 def9a9eaeff9c1c00cc51000d9bbfc197d4c7fa3..4d18a6fe0fe86364bff1a1db5c1ac3ebbaab81d8 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

@@ -303,7 +303,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {

// 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.

- requestContext.startNetLogToFile(file.getPath());

+ requestContext.startNetLogToFile(file.getPath(), false);

// Start a request.

TestUrlRequestListener listener = new TestUrlRequestListener();

@@ -333,7 +333,8 @@ public class CronetUrlRequestContextTest extends CronetTestBase {

getInstrumentation().getTargetContext()));

File file = File.createTempFile("cronet", "json", directory);

try {

- mActivity.mUrlRequestContext.startNetLogToFile(file.getPath());

+ mActivity.mUrlRequestContext.startNetLogToFile(file.getPath(),

+ false);

fail("Should throw an exception.");

} catch (Exception e) {

assertEquals("Context is shut down.", e.getMessage());

@@ -350,10 +351,10 @@ public class CronetUrlRequestContextTest extends CronetTestBase {

getInstrumentation().getTargetContext()));

File file = File.createTempFile("cronet", "json", directory);

// Start NetLog multiple times.

- mActivity.mUrlRequestContext.startNetLogToFile(file.getPath());

+ mActivity.mUrlRequestContext.startNetLogToFile(file.getPath(), false);

// Start a request.

TestUrlRequestListener listener = new TestUrlRequestListener();

UrlRequest urlRequest = mActivity.mUrlRequestContext.createRequest(

@@ -374,7 +375,7 @@ public class CronetUrlRequestContextTest extends CronetTestBase {

File directory = new File(PathUtils.getDataDirectory(

getInstrumentation().getTargetContext()));

File file = File.createTempFile("cronet", "json", directory);

- mActivity.mUrlRequestContext.startNetLogToFile(file.getPath());

+ mActivity.mUrlRequestContext.startNetLogToFile(file.getPath(), false);

// Start a request.

TestUrlRequestListener listener = new TestUrlRequestListener();

UrlRequest urlRequest = mActivity.mUrlRequestContext.createRequest(