Intermittent /css/css-multicol/increase-prev-sibling-height.html | Testing http://web-platform.test:8000/css/css-multicol/increase-prev-sibling-height.html == http://web-platform.test:8000/css/css-multicol/increase-prev-sibling-height-ref.html
Categories
(Core :: Layout: Columns, defect, P5)
Tracking
()
Tracking | Status | |
---|---|---|
firefox89 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: dholbert)
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
Filed by: abutkovits [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=313190010&repo=mozilla-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/UER9X7Z8TEGwLQxKPq4EAw/runs/1/artifacts/public/logs/live_backing.log
Reftest URL: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/UER9X7Z8TEGwLQxKPq4EAw/runs/1/artifacts/public/logs/live_backing.log&only_show_unexpected=1
[task 2020-08-17T10:57:16.357Z] 10:57:16 INFO - TEST-START | /css/css-multicol/increase-prev-sibling-height.html
[task 2020-08-17T10:57:17.211Z] 10:57:17 INFO - TEST-UNEXPECTED-FAIL | /css/css-multicol/increase-prev-sibling-height.html | Testing http://web-platform.test:8000/css/css-multicol/increase-prev-sibling-height.html == http://web-platform.test:8000/css/css-multicol/increase-prev-sibling-height-ref.html
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 34•4 years ago
|
||
This test is annotated as random [PASS, FAIL]
on a bunch of configurations; really, it's probably random everywhere, and we just didn't cover all of the configurations.
Looking at the test, I think it needs reftest-wait
. The reftest snapshot looks like it's been taken before the dynamic JS tweak has taken effect. reftest-wait
would address this and hopefully make the test pass reliably, which would let us remove the .ini annotation file altogether.
Assignee | ||
Comment 35•4 years ago
|
||
This test has some JS that restyles an element, and the test implicitly expects
this JS to run before the screenshot is taken (otherwise the test fails).
However: before this patch, I don't think there's anything stopping the
test-harness from taking its screenshot earlier -- the screenshot can happen as
soon as the load event fires (and after any pending paints at that point are
flushed). This may be before the test's requestAnimationFrame-triggered JS has
been run.
This patch addresses this issue by adding 'reftest-wait', which explicitly
requires the test harness to delay its screenshot until after the restyle has
occurred.
Updated•4 years ago
|
Assignee | ||
Comment 36•4 years ago
|
||
Comment 37•4 years ago
|
||
Comment 39•4 years ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Description
•