Should disable buffer-full-add-after-full-event web platform tests
Categories
(Core :: DOM: Performance APIs, enhancement, P1)
Tracking
()
People
(Reporter: whawkins, Assigned: bdekoz)
Details
Based on the comments in the test source code itself (
// Since we have no strict guarantees when an entry will be added to the
// buffer, waiting till next task to try to avoid flakiness.
) I propose that we disable the buffer-full-add-after-full-event test. Given a recent proposed implementation change (see bug 1525051), we are likely to see this test intermittently pass/fail.
|
Reporter | |
Comment 1•6 years ago
|
||
Suggested that this be fixed upstream:
|
|
Reporter | |
Updated•6 years ago
|
|
|
Reporter | |
Comment 2•6 years ago
|
||
Fix submitted via upstream. Proposed fix can be viewed at https://github.com/web-platform-tests/wpt/pull/16228
|
||
Updated•6 years ago
|
|
||
Comment 3•5 years ago
|
||
(In reply to Will Hawkins from comment #2)
Fix submitted via upstream. Proposed fix can be viewed at https://github.com/web-platform-tests/wpt/pull/16228
Looks like the upstream fix was merged. Should this bug be closed?
|
||
Updated•5 years ago
|
|
|
||
Updated•5 years ago
|
|
|
||
Updated•3 years ago
|
|
||
Comment 4•3 years ago
|
||
Redirect a needinfo that is pending on an inactive user to the triage owner.
:fdoty, since the bug has high priority and recent activity, could you have a look please?
For more information, please visit auto_nag documentation.
|
||
Comment 5•3 years ago
|
||
(In reply to Daniel Holbert [:dholbert] from comment #3)
(In reply to Will Hawkins from comment #2)
Fix submitted via upstream. Proposed fix can be viewed at https://github.com/web-platform-tests/wpt/pull/16228
Looks like the upstream fix was merged. Should this bug be closed?
Marking fixed based on this comment stating fix has been merged :
hawkinsw:whh8b_fix_15849
Description
•