Closed Bug 1680964 Opened 2 years ago Closed 2 years ago

nsCanvasFrame unintentionally pushes fixed pos. placeholder frames in some edge cases

Categories

(Core :: Layout, defect, P2)

defect

Tracking

()

RESOLVED FIXED
85 Branch
Tracking Status
firefox85 --- fixed

People

(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)

References

Details

Attachments

(1 file)

I've discovered that there's a minor error in the code from bug 1665214 here:
https://searchfox.org/mozilla-central/rev/dbc93934c80834ad39dfce25314da2046c1b7a95/layout/generic/nsCanvasFrame.cpp#841-846
We shouldn't push all frames in mFrames, just the kidFrame. If the page has fixed pos. content then we can also have placeholder frames here that should stay.

The bug only occurs when the print fragmentation fallback mechanism is enabled, which is disabled by default currently.

Blocks: 1681052
Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d74502448f09
Don't push all child frames, just 'kidFrame'.  r=TYLin
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
You need to log in before you can comment on or make changes to this bug.