RenderLayer::scrollRectToVisible has three different paths including one for RenderLayers that scroll, one for top-level FrameViews, and one for child FrameViews. The combined complexity of the three paths makes the method quite large and hard to understand. This bug tracks refactoring the logic so that it's easier to understand.
Created attachment 451093 [details] Patch
Created attachment 451109 [details] Patch
Created attachment 451219 [details] Patch
Created attachment 451275 [details] Patch
Created attachment 451362 [details] Patch
Comment on attachment 451362 [details] Patch Okay. After fighting with the CI, I think this patch is ready for review. :)
<rdar://problem/88893284>
Pull request: https://github.com/WebKit/WebKit/pull/1088
Committed 252595@main (ba34b343955e): <https://commits.webkit.org/252595@main> Reviewed commits have been landed. Closing PR #1088 and removing active labels.