Bug 205709 - Reformat WebPage logging
Summary: Reformat WebPage logging
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Keith Rollin
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-01-02 20:19 PST by Keith Rollin
Modified: 2020-01-06 12:46 PST (History)
4 users (show)

See Also:


Attachments
Patch (18.12 KB, patch)
2020-01-02 20:49 PST, Keith Rollin
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Keith Rollin 2020-01-02 20:19:49 PST
Update the format used by WebPage in its RELEASE_LOG logging. Use the format used by WebPageProxy and NetworkResourceLoader, which is generally of the form:

    <object-address> - [<values that help thread together operations>] <class>::<method>: <message and other useful values>

So, for example:

    0x4a1df5000 - WebLoaderStrategy::scheduleLoad: Resource is being scheduled with the NetworkProcess (frame=0x4a1db0220, priority=0, webPageID=15, frameID=3, resourceID=32)',

becomes:

    0x4a1df5000 - [resourceLoader=0x1418b7200, frameLoader=0x1326d7340, frame=0x4a1db0220, webPageID=15, frameID=3, resourceID=32] WebLoaderStrategy::scheduleLoad: Resource is being scheduled with the NetworkProcess (priority=2)
Comment 1 Radar WebKit Bug Importer 2020-01-02 20:19:58 PST
<rdar://problem/58290285>
Comment 2 Keith Rollin 2020-01-02 20:49:22 PST
Created attachment 386656 [details]
Patch
Comment 3 Brent Fulgham 2020-01-06 12:05:04 PST
Comment on attachment 386656 [details]
Patch

Much tidier! r=me
Comment 4 WebKit Commit Bot 2020-01-06 12:46:46 PST
Comment on attachment 386656 [details]
Patch

Clearing flags on attachment: 386656

Committed r254078: <https://trac.webkit.org/changeset/254078>
Comment 5 WebKit Commit Bot 2020-01-06 12:46:52 PST
All reviewed patches have been landed.  Closing bug.