1531470 - Support position:sticky in OOP iframes

Status: RESOLVED FIXED

(Core :: Layout, enhancement, P3)

Description

[Jonathan Watt [:jwatt]]

There may be work we need to do to properly support position:sticky in OOP iframes. For example, IsStickyFrameActive in nsDisplayList.cpp using nsLayoutUtils::GetCrossDocParentFrame to walk up across document boundaries in a way that will not be possible in OOP iframes.

Comment 1

[Hiroyuki Ikezoe (:hiro)]

position:sticky should not stick to a scrollable frame in an ancestor document, I think. We should specify SCROLLABLE_SAME_DOC and SCROLLABLE_INCLUDE_HIDDEN flags just like we do in StickyScrollContainer.

Comment 2

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1531470 - Use nsLayoutUtils::GetNearestScrollableFrame with SAME_DOC and INCLUDE_HIDDEN flags to get position:sticky target scrollable frame. r?kats

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4f78b47f52b2
Use nsLayoutUtils::GetNearestScrollableFrame with SAME_DOC and INCLUDE_HIDDEN flags to get position:sticky target scrollable frame. r=kats

Comment 4

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/4f78b47f52b2

Comment 5

[Chris Peterson [:cpeterson]]

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