Bug 162237

Summary: Add new logging for network resource loading
Product: WebKit Reporter: Keith Rollin <krollin>
Component: WebKit2Assignee: Keith Rollin <krollin>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, dbates, japhet, koivisto
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 162235    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch none

Keith Rollin
Reported 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.
Attachments
Patch (1.80 KB, patch)
2016-09-19 14:35 PDT, Keith Rollin 		no flags
DetailsFormatted DiffDiff
Patch (29.38 KB, patch)
2016-09-20 15:12 PDT, Keith Rollin 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Keith Rollin
Comment 1 2016-09-19 14:35:56 PDT
Created attachment 289258 [details] Patch
Keith Rollin
Comment 2 2016-09-19 14:36:57 PDT
I uploaded the wrong patch.
Keith Rollin
Comment 3 2016-09-20 15:12:13 PDT
Created attachment 289403 [details] Patch
WebKit Commit Bot
Comment 4 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>
WebKit Commit Bot
Comment 5 2016-09-20 17:29:35 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.