RESOLVED FIXED 256537
[Repaint] Cleanup RenderElement::repaintAfterLayoutIfNeeded
https://bugs.webkit.org/show_bug.cgi?id=256537
Summary [Repaint] Cleanup RenderElement::repaintAfterLayoutIfNeeded
zalan
Reported 2023-05-09 11:36:39 PDT
ssia
Attachments
Patch (19.72 KB, patch)
2023-05-09 12:21 PDT, zalan
no flags
Patch (19.61 KB, patch)
2023-05-09 12:29 PDT, zalan
ews-feeder: commit-queue-
Patch (19.69 KB, patch)
2023-05-09 16:01 PDT, zalan
no flags
zalan
Comment 1 2023-05-09 12:21:31 PDT
zalan
Comment 2 2023-05-09 12:29:16 PDT
Simon Fraser (smfr)
Comment 3 2023-05-09 14:07:59 PDT
Comment on attachment 466297 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=466297&action=review > Source/WebCore/rendering/RenderElement.cpp:1287 > + auto width = absoluteValue(newOutlineAndBoxShadowBox.width() - oldOutlineAndBoxShadowBox.width()); Nicer to compute a size delta (with "delta" in the name).
zalan
Comment 4 2023-05-09 15:24:11 PDT
(In reply to Simon Fraser (smfr) from comment #3) > Comment on attachment 466297 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=466297&action=review > > > Source/WebCore/rendering/RenderElement.cpp:1287 > > + auto width = absoluteValue(newOutlineAndBoxShadowBox.width() - oldOutlineAndBoxShadowBox.width()); > > Nicer to compute a size delta (with "delta" in the name). Good point!
zalan
Comment 5 2023-05-09 16:01:27 PDT
EWS
Comment 6 2023-05-09 19:32:33 PDT
Committed 263890@main (e10f145ce649): <https://commits.webkit.org/263890@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 466300 [details].
Radar WebKit Bug Importer
Comment 7 2023-05-09 19:33:19 PDT
Note You need to log in before you can comment on or make changes to this bug.