[webkit-reviews] review requested: [Bug 78315] REGRESSION(99539): Infinite repaint loop with SVGImage and deferred repaint timers : [Attachment 127484] revised patch

[bugzilla-daemon at webkit.org]

Tim Horton <timothy_horton at apple.com> has asked  for review:
Bug 78315: REGRESSION(99539): Infinite repaint loop with SVGImage and deferred
repaint timers
https://bugs.webkit.org/show_bug.cgi?id=78315

Attachment 127484: revised patch
https://bugs.webkit.org/attachment.cgi?id=127484&action=review

------- Additional Comments from Tim Horton <timothy_horton at apple.com>
The logic was slightly wrong; before, with the !timer.isActive() inside the
same condition as needsLayout(), if the timer was active and we were in layout,
we would (bad!) perform an immediate repaint. Uploading in hopes that the
Chromium EWS is happier with this, but this does fix it locally.