Bug 192872 - Improve release-level page-load logging
Summary: Improve release-level page-load 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: 2018-12-19 13:19 PST by Keith Rollin
Modified: 2018-12-21 12:23 PST (History)
8 users (show)

See Also:


Attachments
Patch (102.25 KB, patch)
2018-12-19 13:27 PST, Keith Rollin
no flags Details | Formatted Diff | Diff
Patch (101.21 KB, patch)
2018-12-19 13:55 PST, Keith Rollin
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews107 for mac-sierra-wk2 (3.29 MB, application/zip)
2018-12-19 15:02 PST, EWS Watchlist
no flags Details
Archive of layout-test-results from ews103 for mac-sierra (2.42 MB, application/zip)
2018-12-19 15:08 PST, EWS Watchlist
no flags Details
Archive of layout-test-results from ews115 for mac-sierra (2.19 MB, application/zip)
2018-12-19 15:32 PST, EWS Watchlist
no flags Details
Patch (101.15 KB, patch)
2018-12-19 17:02 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 2018-12-19 13:19:09 PST
There are a number of reported bugs that are difficult or impossible to track down with our current level of logging. Additionally, some software groups lower in the page-loading stack have requested logging sufficient for tracking a user-visible error message down to the requested resource that caused the message. Add more-comprehensive logging to address these issues/requests.
Comment 1 Radar WebKit Bug Importer 2018-12-19 13:19:38 PST
<rdar://problem/46850309>
Comment 2 Keith Rollin 2018-12-19 13:27:34 PST
Created attachment 357713 [details]
Patch
Comment 3 Keith Rollin 2018-12-19 13:55:33 PST
Created attachment 357718 [details]
Patch
Comment 4 EWS Watchlist 2018-12-19 15:02:01 PST
Comment on attachment 357718 [details]
Patch

Attachment 357718 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/10478272

New failing tests:
http/wpt/html/semantics/text-level-semantics/the-a-element/a-download-click-404.html
imported/w3c/web-platform-tests/html/semantics/text-level-semantics/the-a-element/a-download-click.html
imported/w3c/web-platform-tests/html/semantics/embedded-content/the-area-element/area-download-click.html
imported/w3c/web-platform-tests/html/semantics/text-level-semantics/the-a-element/a-download-click-404.html
fast/loader/form-submission-after-beforeunload-cancel.html
Comment 5 EWS Watchlist 2018-12-19 15:02:03 PST
Created attachment 357731 [details]
Archive of layout-test-results from ews107 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 6 EWS Watchlist 2018-12-19 15:08:13 PST
Comment on attachment 357718 [details]
Patch

Attachment 357718 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/10478462

New failing tests:
fast/loader/form-submission-after-beforeunload-cancel.html
Comment 7 EWS Watchlist 2018-12-19 15:08:15 PST
Created attachment 357734 [details]
Archive of layout-test-results from ews103 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 8 EWS Watchlist 2018-12-19 15:32:26 PST
Comment on attachment 357718 [details]
Patch

Attachment 357718 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/10478274

New failing tests:
fast/loader/form-submission-after-beforeunload-cancel.html
Comment 9 EWS Watchlist 2018-12-19 15:32:27 PST
Created attachment 357740 [details]
Archive of layout-test-results from ews115 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 10 Chris Dumez 2018-12-19 16:51:32 PST
Comment on attachment 357718 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=357718&action=review

> Source/WebCore/loader/FrameLoader.cpp:3361
> +            shouldClose());

This is not safe to call for logging, this fires events.
Comment 11 Keith Rollin 2018-12-19 17:02:25 PST
Created attachment 357760 [details]
Patch
Comment 12 WebKit Commit Bot 2018-12-20 11:23:28 PST
Comment on attachment 357760 [details]
Patch

Clearing flags on attachment: 357760

Committed r239457: <https://trac.webkit.org/changeset/239457>
Comment 13 WebKit Commit Bot 2018-12-20 11:23:30 PST
All reviewed patches have been landed.  Closing bug.
Comment 14 Keith Rollin 2018-12-21 12:23:34 PST
This patch caused a crash in WebResourceLoader::willSendRequest.

Fixed in Bug 192989.