Bug 162237 - Add new logging for network resource loading
Summary: Add new logging for network resource loading
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Keith Rollin
URL:
Keywords:
Depends on: 162235
Blocks:
  Show dependency tree / graph
 
Reported: 2016-09-19 14:23 PDT by Keith Rollin
Modified: 2016-09-20 17:29 PDT (History)
5 users (show)

See Also:

Attachments
Patch (1.80 KB, patch)
2016-09-19 14:35 PDT, Keith Rollin 		no flags Details | Formatted Diff | Diff
Patch (29.38 KB, patch)
2016-09-20 15:12 PDT, Keith Rollin 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Keith Rollin 2016-09-19 14:23:44 PDT 
Add new logging to better track the downloading of network resources. This largely involves adding logging along the paths less travelled. The previous logging would log the normal path executed when downloading resources, but experience is showing us that this is not sufficient. Sometimes logging would suddenly stop, indicating either that some non-normal path was taken, or that the process was hung. By adding more logging, we should be able to identify the first case.

For the most part, there should be little-to-no new logging along the main path, meaning that the overall impact of our logging on the system buffers and sysdiagnose should be minimal.
Comment 1 Keith Rollin 2016-09-19 14:35:56 PDT 
Created attachment 289258 [details]
Patch
Comment 2 Keith Rollin 2016-09-19 14:36:57 PDT 
I uploaded the wrong patch.
Comment 3 Keith Rollin 2016-09-20 15:12:13 PDT 
Created attachment 289403 [details]
Patch
Comment 4 WebKit Commit Bot 2016-09-20 17:29:30 PDT 
Comment on attachment 289403 [details]
Patch

Clearing flags on attachment: 289403

Committed r206189: <http://trac.webkit.org/changeset/206189>
Comment 5 WebKit Commit Bot 2016-09-20 17:29:35 PDT 
All reviewed patches have been landed.  Closing bug.