RESOLVED FIXED 13544
REGRESSION (r21120): Member profile frame not erased at Yahoo! Answers
https://bugs.webkit.org/show_bug.cgi?id=13544
Summary REGRESSION (r21120): Member profile frame not erased at Yahoo! Answers
mitz
Reported 2007-04-29 15:16:58 PDT
Repaint regression, probably introduced in <http://trac.webkit.org/projects/webkit/changeset/21120>: open the URL and hover over a member's avatar and wait for the member info box to appear, then move the mouse away to make the box disappear. Three sides of the gray frame remain visible until you force a repaint.
Attachments
Reduction (576 bytes, text/html)
2007-04-29 16:14 PDT, mitz
no flags
Repaint layers becoming invisible even if they also need layout (10.41 KB, patch)
2007-04-29 16:39 PDT, mitz
hyatt: review+
mitz
Comment 1 2007-04-29 16:14:37 PDT
Created attachment 14271 [details] Reduction Layers need to repaint when turning invisible.
mitz
Comment 2 2007-04-29 16:39:42 PDT
Created attachment 14272 [details] Repaint layers becoming invisible even if they also need layout Includes change log and repaint test.
Dave Hyatt
Comment 3 2007-04-29 17:41:36 PDT
Random unrelated question, but why did your previous change remove a setNeedsLayout(false) call? Was it redundant?
Dave Hyatt
Comment 4 2007-04-29 17:43:25 PDT
Comment on attachment 14272 [details] Repaint layers becoming invisible even if they also need layout r=me
Mark Rowe (bdash)
Comment 5 2007-04-29 20:49:35 PDT
Landed in r21186.
mitz
Comment 6 2007-04-29 22:42:53 PDT
(In reply to comment #3) > Random unrelated question, but why did your previous change remove a > setNeedsLayout(false) call? Was it redundant? > layoutBlockChildren uses the logic that "we don't need to repaint children if the parent needs layout, because in that case it will do a full repaint". But by resetting the bit in layoutBlockChildren it was invalidating this logic since by the time layoutBlock called repaintAfterLayoutIfNeeded, the latter couldn't tell that it needed to do a full repaint. I think I explained it in my ChangeLog ;-) and it's covered by one of the repaint tests IIRC.
Dave Hyatt
Comment 7 2007-04-29 22:58:01 PDT
Reading ChangeLogs is hard.
Note You need to log in before you can comment on or make changes to this bug.