[webkit-reviews] review granted: [Bug 188122] [WK1] ASSERTION FAILED: renderer().repaintLayoutRects().m_repaintRect == renderer().clippedOverflowRectForRepaint(renderer().containerForRepaint()) in WebCore::RenderLayer::updateLayerPositionsAfterScroll : [Attachment 345949] Patch

[bugzilla-daemon at webkit.org]

Simon Fraser (smfr) <simon.fraser at apple.com> has granted zalan
<zalan at apple.com>'s request for review:
Bug 188122: [WK1] ASSERTION FAILED:
renderer().repaintLayoutRects().m_repaintRect ==
renderer().clippedOverflowRectForRepaint(renderer().containerForRepaint()) in
WebCore::RenderLayer::updateLayerPositionsAfterScroll
https://bugs.webkit.org/show_bug.cgi?id=188122

Attachment 345949: Patch

https://bugs.webkit.org/attachment.cgi?id=345949&action=review




--- Comment #2 from Simon Fraser (smfr) <simon.fraser at apple.com> ---
Comment on attachment 345949
  --> https://bugs.webkit.org/attachment.cgi?id=345949
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=345949&action=review

> Source/WebCore/ChangeLog:9
> +	   We can't reliably tell if repaint rects need to be recomputed when
FrameView's m_paintsEntireContents flag flips due to layer backing changes.

Maybe expand this comment a bit.