[webkit-reviews] review requested: [Bug 5699] REGRESSION: incomplete repaint : [Attachment 5279] Possible fix

[bugzilla-request-daemon at opendarwin.org]

opendarwin.org at mitzpettel.com has asked  for review:
Bug 5699: REGRESSION: incomplete repaint
http://bugzilla.opendarwin.org/show_bug.cgi?id=5699

Attachment 5279: Possible fix
http://bugzilla.opendarwin.org/attachment.cgi?id=5279&action=edit

------- Additional Comments from opendarwin.org at mitzpettel.com
RenderObject::repaintAfterLayoutIfNeeded was lying about whether it did a full
repaint. This patch makes it return the truth. The return value is only used in
RenderBlock::layoutBlock.

An alternative approach (perhaps better performance-wise) is to let
repaintAfterLayoutIfNeeded take an extra "must repaint" rect as an argument and
assure that it was repainted as well (if it returned true).

If/when the code change is approved, I'll submit a more commit-friendly patch.
I don't think there can be an automatic test for such a rendering bug.