New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 767067 link

Starred by 2 users
Status: Fixed
Owner:
█ sgu...@chromium.org
Last visit > 30 days ago
Closed: Sep 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Add net-log to Android Webview

Project Member Reported by sgu...@chromium.org, Sep 20 2017 
WebView does not have net-log. This is to add support in command line as a basic developer feature.

The command line version will only work in user-debug devices as there is no way to set it up in user devices.

as a future idea, one can integrate it with chrome internals UI perhaps, so that people can collect information from their browsers (just like how chrome dev tools work). This is outside the scope of this bug.

Since webview does not have a good signal to shutdown net-log, we will be ok with truncated files. Dealing with this is also outside the scope of this bug and can be done as part of the future idea above.
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 20 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f9d7b1b1b968328b10da7c4a193c4dd8f25e0b7d

commit f9d7b1b1b968328b10da7c4a193c4dd8f25e0b7d
Author: Selim Gurun <sgurun@google.com>
Date: Wed Sep 20 19:39:08 2017

Add net-log to WebView for easier network debugging

The lack of net-log makes it harder to debug network issues. This
CL adds net-log support. The net-log files are created in apps
data directory with default name net_log.json or whatever
is provided in the command line.

webView net_log can only be enabled via command line flag log-net-log


Bug:  767067 
Change-Id: Ib463e8767eb400a61e5013f82ace6b1c0b212fe1
Reviewed-on: https://chromium-review.googlesource.com/674084
Commit-Queue: Selim Gurun <sgurun@chromium.org>
Reviewed-by: Nate Fischer <ntfschr@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#503227}
[modify] https://crrev.com/f9d7b1b1b968328b10da7c4a193c4dd8f25e0b7d/android_webview/browser/net/aw_url_request_context_getter.cc
[modify] https://crrev.com/f9d7b1b1b968328b10da7c4a193c4dd8f25e0b7d/android_webview/browser/net/aw_url_request_context_getter.h

Comment 2 by sgu...@chromium.org, Sep 20 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment