The proposed changes are related to bug 229748.
Created attachment 442641 [details] Patch
Comment on attachment 442641 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=442641&action=review > LayoutTests/fast/shadow-dom/slot-removal-same-name.html:1 > +<script> Missing DOCTYPE? > LayoutTests/fast/shadow-dom/slot-removal-same-name.html:3 > + let shadow0 = document.body.attachShadow({mode: 'open'}); Can we call this just shadowRoot? Also, use const since we're reassigning another value to it. > LayoutTests/fast/shadow-dom/slot-removal-same-name.html:6 > + document.body.offsetTop; document.body.getBoundingClientRect() is more sure way of ensuring updating the layout. offsetTop can be optimized in some cases.
Created attachment 442644 [details] Patch
Address feedback comments.
Committed r284969 (243620@main): <https://commits.webkit.org/243620@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 442644 [details].
<rdar://problem/84738513>