RESOLVED FIXED 13391
REGRESSION (r18819): Incomplete repaint of button text 
https://bugs.webkit.org/show_bug.cgi?id=13391
Summary REGRESSION (r18819): Incomplete repaint of button text
mitz
Reported Wednesday, April 18, 2007 2:43:57 PM UTC
See the attached test case.
Attachments
Test case (208 bytes, text/html)
2007-04-18 06:44 PDT, mitz 		no flags
Details
Just ignore lightweight clip in repaint rect computation (39.58 KB, patch)
2007-04-21 08:53 PDT, mitz 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 Wednesday, April 18, 2007 2:44:30 PM UTC
Created attachment 14073 [details] Test case
mitz
Comment 2 Wednesday, April 18, 2007 2:45:18 PM UTC
This bug affects the Web Inspector (at least on trunk).
mitz
Comment 3 Wednesday, April 18, 2007 3:44:48 PM UTC
Corrected the revision range of the regression. The later changes merely made the issue visible in the inspector.
mitz
Comment 4 Wednesday, April 18, 2007 4:27:34 PM UTC
This bug is a regression from Safari 2.0.4 since it also affects <button>. The regression was caused by <http://trac.webkit.org/projects/webkit/changeset/18819>. The RenderBox::computeAbsoluteRepaintRect() logic for the hasControlClip() case is very strange - it doesn't look at the container o at all. I think it should intersect with o's controlClipRect(). However, in the case of this bug, the controlClipRect() is invalid since the container is in mid-layout (exactly the same problem discussed in the comments for the hasOverflowClip() case).
mitz
Comment 5 Saturday, April 21, 2007 4:53:02 PM UTC
Created attachment 14122 [details] Just ignore lightweight clip in repaint rect computation I think it's okay for now to just ignore the clip. Typically the contents don't overflow the control anyway, so this shouldn't cause successive repainting (for popups and listboxes, the contents are completely controlled by the engine).
Darin Adler
Comment 6 Saturday, April 21, 2007 6:15:56 PM UTC
Comment on attachment 14122 [details] Just ignore lightweight clip in repaint rect computation r=me; seems OK to ignore the clipping when repainting
Sam Weinig
Comment 7 Sunday, April 22, 2007 4:58:55 AM UTC
Landed in r21002.
Dave Hyatt
Comment 8 Sunday, April 22, 2007 6:11:03 AM UTC
Please file a followup bug to track the FIXME added to the code.
mitz
Comment 9 Sunday, April 22, 2007 6:50:40 AM UTC
(In reply to comment #8) > Please file a followup bug to track the FIXME added to the code. > Bug 13443.
Note You need to log in before you can comment on or make changes to this bug.