Make the fission subtests for browser_test_group_fission work on non-WR
Categories
(Core :: Panning and Zooming, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
In bug 1559259 :rhunt added a fix for the APZ/fission hit-testing codepath for non-WR. This should have made all the subtests pass without WebRender, but for some reason they were still failing (see e.g. bug 1559565). We should figure out why and fix it.
One possible reason is that the OOP iframe doesn't get a displayport and so APZ doesn't have a layers id for it. That is, the rectangle is just in the dispatch-to-content region for the top-level content document. If that's the case then we can probably modify the test to force setting a displayport. But it might also get fixed by bug 1541589.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
I moved the tests to run on non-WR as well and ran with --verify
a few times on local Windows and Linux builds, and it seems to pass perfectly fine. Maybe something else fixed it along the way.
Try push shows it failing on windows 7 but passing on linux and windows 10 (and macOS). So maybe we can just disable the test on win7 and move the subtests unconditionally.
Assignee | ||
Comment 2•5 years ago
|
||
There isn't a mochitest condition for "win7" specifically, but it's the only windows 32-bit platform, so we can condition on that. Try push: https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&revision=6bbdeebb26d6f125b7734e83637fb1d2d8626021
Assignee | ||
Comment 3•5 years ago
|
||
This also disables the test on Windows 7 because the newly enabled subtests
fail intermittently there. We don't care so much about Windows 7 these days,
and I don't have a local setup to reproduce it, so I didn't investigate the
failure.
Assignee | ||
Updated•5 years ago
|
Comment 5•5 years ago
|
||
bugherder |
Comment 6•5 years ago
|
||
Retroactively moving fixed bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to an appropriate Fission Milestone.
This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:
0ee3c76a-bc79-4eb2-8d12-05dc0b68e732
Description
•