Summary: | Layout Test http/wpt/beacon/beacon-async-error-logging.html is flaky | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Matt Lewis <jlewis3> | ||||||||
Component: | Tools / Tests | Assignee: | Chris Dumez <cdumez> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | achristensen, cdumez, commit-queue, lforschler, rniwa, ryanhaddad, webkit-bug-importer, youennf | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Matt Lewis
2018-01-26 13:11:46 PST
Don't worry about the regression point. The test currently relies on a setTimeout. I'll fix. Created attachment 332430 [details]
WIP Patch
Created attachment 332449 [details]
Patch
Comment on attachment 332449 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=332449&action=review > LayoutTests/http/wpt/beacon/beacon-async-error-logging.html:10 > + internals.setConsoleMessageListener(function() { Can we make it sort of working on regular environment by testing for window.internals and adding a text message stating to look for console message? Created attachment 332451 [details]
Patch
Comment on attachment 332451 [details] Patch Clearing flags on attachment: 332451 Committed r227712: <https://trac.webkit.org/changeset/227712> All reviewed patches have been landed. Closing bug. |