Closed
Bug 1510639
Opened 6 years ago
Closed 6 years ago
Add reftests that exericse different sticky-position scenarios wrt enclosing scrollframe and reference frames
Categories
(Core :: Graphics: WebRender, enhancement, P2)
Tracking
()
RESOLVED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
(Whiteboard: [gfx-noted])
Attachments
(1 file)
Spinoff from https://phabricator.services.mozilla.com/D13081#323451
Bug 1503447 made some changes to get rid of the stacking context origin property passed to WR. However in order to pull this off it had to twist around some of the sticky positioning handling code on the gecko side. In doing so we discovered that there are scenarios that our existing reftests don't cover, so we should add explicit tests for them.
In particular we should add a reftests that cover the scenarios at [1] and [2] (4 reftests in total, there's two per URL). The first two exercise the case where there's a sticky item inside a transformed element inside a scrollframe. The next two exercise the case where there's a sticky item inside a scrollframe inside a transformed element.
[1] https://mozilla.staktrace.com/bug/1503447-1.html
[2] https://mozilla.staktrace.com/bug/1503447-2.html
Assignee | ||
Updated•6 years ago
|
Summary: Add reftest for case where sticky item and enclosing scrollframe share the same transformed reference frame → Add reftests that exericse different sticky-position scenarios wrt enclosing scrollframe and reference frames
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Whoops, the failure on Linux/Android is because the scrollbars don't cover the text like they do on win/mac. Updated try push:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e45b40af424825cc24eb2685b452ccc045cca03e
I'm expecting this to have one Android failure which is fuzzable. I just need to see the failure to know the fuzz values. Rest should be green.
Updated•6 years ago
|
Priority: -- → P2
Assignee | ||
Comment 3•6 years ago
|
||
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e79c6da7b0fe
Add reftests for various scenarios involving sticky items inside scrollframes and transforms. r=kvark
Comment 5•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in
before you can comment on or make changes to this bug.
Description
•