286002 2025-01-15 08:43:28 -0800 REGRESSION (286055@main): Relative-positioned input elements in scroll areas don't render outlines 2025-01-20 09:41:52 -0800 1 1 1 Unclassified WebKit Compositing WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=282478 InRadar P2 Normal --- 1 feaneron simon.fraser simon.fraser webkit-bug-importer oldest_to_newest 2087465 0 473907 feaneron 2025-01-15 08:43:28 -0800 Created attachment 473907 Reproducer This seems to be a regression introduced by https://commits.webkit.org/286055@main for fixing https://bugs.webkit.org/show_bug.cgi?id=282478 See the attachment for a reproducer. Specifically, commit https://commits.webkit.org/286055@main introduces the following line: ``` if (localPaintFlags.contains(PaintLayerFlag::PaintingOverflowContents)) return false; ``` and this is preventing the outline from the input field from rendering. 2087552 1 webkit-bug-importer 2025-01-15 13:30:45 -0800 <rdar://problem/142995142> 2087745 2 feaneron 2025-01-16 06:43:33 -0800 Pull request: https://github.com/WebKit/WebKit/pull/39130 2088167 3 simon.fraser 2025-01-17 16:02:16 -0800 Seems to happen when the <input> itself is `position:relative` (but a relative container is OK). 2088171 4 simon.fraser 2025-01-17 16:30:04 -0800 The issue here is that we want to avoid painting the outline *for the scroller itself* in the scrolled contents layer (hence the `return false` for PaintingOverflowContents), but we do need to paint the outline in any descendant layers. 2088211 5 simon.fraser 2025-01-17 18:11:33 -0800 I have an alternative fix. 2088215 6 simon.fraser 2025-01-17 18:17:21 -0800 Pull request: https://github.com/WebKit/WebKit/pull/39229 2088534 7 ews-feeder 2025-01-20 09:41:49 -0800 Committed 289154@main (d183e4fe4bcd): <https://commits.webkit.org/289154@main> Reviewed commits have been landed. Closing PR #39229 and removing active labels. 473907 2025-01-15 08:43:28 -0800 2025-01-15 08:43:28 -0800 Reproducer outline-issue.html text/html 2046 feaneron PCFET0NUWVBFIGh0bWw+CjxodG1sIGxhbmc9ImVuIj4KPGhlYWQ+CiAgPG1ldGEgY2hhcnNldD0i VVRGLTgiPgogIDxtZXRhIG5hbWU9InZpZXdwb3J0IiBjb250ZW50PSJ3aWR0aD1kZXZpY2Utd2lk dGgsIGluaXRpYWwtc2NhbGU9MS4wIj4KICA8dGl0bGU+RG9jdW1lbnQ8L3RpdGxlPgogIDxzdHls ZT4KICAjcGFuZWx0ZXN0MSB7CiAgICBtYXgtaGVpZ2h0OiAzMDBweDsKICAgIG92ZXJmbG93OiBz Y3JvbGw7CiAgICBib3JkZXI6IDRweCBzb2xpZCBibHVlOwogIH0KICAjcGFuZWx0ZXN0MiB7CiAg ICBtYXgtaGVpZ2h0OiA0MDBweDsKICAgIG92ZXJmbG93OiBzY3JvbGw7CiAgICBib3JkZXI6IDRw eCBzb2xpZCBncmVlbjsKICB9CgogIC5ibG9iIHsKICAgIG1pbi1oZWlnaHQ6IDIzMHB4OwogICAg YmFja2dyb3VuZC1jb2xvcjogZGFya3NsYXRlZ3JleTsKICB9CgogIC5jb250ZW50IHsKICAgIGRp c3BsYXk6IGZsZXg7CiAgICBmbGV4LWRpcmVjdGlvbjogY29sdW1uOwogIH0KCiAgLmlucHV0LWNv bnRhaW5lciB7CiAgICBwb3NpdGlvbjogcmVsYXRpdmU7CiAgICBkaXNwbGF5OiBpbmxpbmUtZmxl eDsKICAgIGJvcmRlci1zdHlsZTogc29saWQ7CiAgICBib3JkZXItd2lkdGg6IDFweDsKICAgIGJh Y2tncm91bmQtY29sb3I6IGRhcmtncmV5OwogICAgYm9yZGVyLWNvbG9yOiBibHVlOwogICAgaGVp Z2h0OiAyLjVyZW07CiAgICBib3gtc2l6aW5nOiBib3JkZXItYm94OwogICAgd2lkdGg6IGluaGVy aXQ7CiAgICBvdXRsaW5lOiAwLjE4NzVyZW0gc29saWQgdHJhbnNwYXJlbnQ7CiAgICBvdXRsaW5l LW9mZnNldDogMC4wNjI1cmVtOwogICAgdHJhbnNpdGlvbjogb3V0bGluZSAwLjE1czsKICAgIG1h cmdpbjogMXJlbTsKICAgIG1heC13aWR0aDogMjUwcHg7CiAgfQoKICBpbnB1dCB7CiAgICBwb3Np dGlvbjogcmVsYXRpdmU7CiAgICBkaXNwbGF5OiBibG9jazsKICAgIGZvbnQtc2l6ZTogMXJlbTsK ICAgIG91dGxpbmU6IG5vbmU7CiAgICBsaW5lLWhlaWdodDogMS4zNzVyZW07CiAgICBwYWRkaW5n OiAwIDAuNzVyZW07CiAgICBib3JkZXI6IG5vbmU7CiAgICBiYWNrZ3JvdW5kLWNvbG9yOiB0cmFu c3BhcmVudDsKICAgIGNvbG9yOiBsaWdodGdyYXk7CiAgICBib3gtc2l6aW5nOiBib3JkZXItYm94 OwogICAgd2lkdGg6IDEwMCU7CgogIH0KCiAgLmlucHV0LWNvbnRhaW5lcjpoYXMoLmlucHV0OmZv Y3VzKTpub3QoOmhhcyguaW5wdXQ6cmVhZC1vbmx5KSkgewogICAgb3V0bGluZS1jb2xvcjogcmVk OwogIH0KICA8L3N0eWxlPgo8L2hlYWQ+Cjxib2R5PgogIDxkaXYgaWQ9InBhbmVsdGVzdDEiIHNs b3Q9ImNvbnRlbnQiPgogICAgPGRpdiBjbGFzcz0iYmxvYiI+TWF4aW11bSBoZWlnaHQgbGltaXRl ZCB0byAzMDBweCB0byBmb3JjZSBvdmVyZmxvdzwvZGl2PgogICAgPGRpdiBjbGFzcz0iY29udGVu dCI+CiAgICAgIDxkaXYgcGFydD0iY29udGFpbmVyIiBjbGFzcz0iaW5wdXQtY29udGFpbmVyIj4K ICAgICAgICA8aW5wdXQgY2xhc3M9ImlucHV0IiAvPgogICAgICA8L2Rpdj4KICAgICAgPGRpdiBw YXJ0PSJjb250YWluZXIiIGNsYXNzPSJpbnB1dC1jb250YWluZXIiPgogICAgICAgIDxpbnB1dCBj bGFzcz0iaW5wdXQiIC8+CiAgICAgIDwvZGl2PgogICAgPC9kaXY+CiAgPC9kaXY+CiAgPGRpdiBp ZD0icGFuZWx0ZXN0MiIgc2xvdD0iY29udGVudCI+CiAgICA8ZGl2IGNsYXNzPSJibG9iIj5NYXhp bXVtIGhlaWdodCBsaW1pdGVkIHRvIDQwMHB4LCB3aGljaCBpcyBtb3JlIHRoYW4gcmVxdWlyZWQs IG5vIG92ZXJmbG93IDwvZGl2PgogICAgPGRpdiBjbGFzcz0iY29udGVudCI+CiAgICAgIDxkaXYg cGFydD0iY29udGFpbmVyIiBjbGFzcz0iaW5wdXQtY29udGFpbmVyIj4KICAgICAgICA8aW5wdXQg Y2xhc3M9ImlucHV0IiAvPgogICAgICA8L2Rpdj4KICAgICAgPGRpdiBwYXJ0PSJjb250YWluZXIi IGNsYXNzPSJpbnB1dC1jb250YWluZXIiPgogICAgICAgIDxpbnB1dCBjbGFzcz0iaW5wdXQiIC8+ CiAgICAgIDwvZGl2PgogICAgPC9kaXY+CiAgPC9kaXY+CjwvYm9keT4KPC9odG1sPiAK