Bug 118635

Summary: Fixed position elements should layout relative to transformed container
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Layout and RenderingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: ahmad.saleem792, ap, bdakin, bfulgham, dino, hyatt, jchaffraix, simon.fraser, thorton, webkit-bug-importer, zalan
Priority: P2 Keywords: BlinkMergeCandidate, InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Test case shows that we have a bug none

Description Ryosuke Niwa 2013-07-12 22:59:25 PDT
Consider merging https://chromium.googlesource.com/chromium/blink/+/f052baa7402aa050a3e5c633a4b374ecf7162926

The original layer coordinates conversion code contains a bug that it forgets
to return for the simple case where the fixed position container found is
already the ancestor layer wanted, and the code falls through to the special
path intended for RenderFlowThread.
Comment 2 Simon Fraser (smfr) 2022-08-22 12:20:37 PDT
I think our containing block logic is correct, but there's still a FIXME for this in compositing code.
Comment 3 Simon Fraser (smfr) 2022-08-22 15:53:57 PDT
Created attachment 461806 [details]
Test case shows that we have a bug
Comment 4 Simon Fraser (smfr) 2022-08-22 15:59:34 PDT
    // FIXME: Handle fixed inside of a transform, which should not behave as fixed.
Comment 5 Simon Fraser (smfr) 2022-08-22 16:46:09 PDT
Pull request: https://github.com/WebKit/WebKit/pull/3560
Comment 6 EWS 2022-08-25 23:48:22 PDT
Committed 253809@main (546a1c950d42): <https://commits.webkit.org/253809@main>

Reviewed commits have been landed. Closing PR #3560 and removing active labels.
Comment 7 Radar WebKit Bug Importer 2022-08-25 23:49:16 PDT
<rdar://problem/99174247>