Bug 9669

Summary: Incomplete repaint when changing an inline's border
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ian
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://meme.b9.com/start.html
Attachments:
Description Flags
Test case
none
Patch
none
Patch including change log and a repaint test hyatt: review+

mitz
Reported 2006-06-30 10:12:32 PDT
When you roll over links in the URL (for example in the top menu bar), they should get a black border on all 4 sides, but the top border is not painted.
Attachments
Test case (168 bytes, text/html)
2006-06-30 10:13 PDT, mitz 		no flags
Details
Patch (10.15 KB, patch)
2006-07-07 09:29 PDT, mitz 		no flags
DetailsFormatted DiffDiff
Patch including change log and a repaint test (57.98 KB, patch)
2006-07-08 11:08 PDT, mitz 		hyatt: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-06-30 10:13:11 PDT
Created attachment 9109 [details] Test case
mitz
Comment 2 2006-07-07 09:29:33 PDT
Created attachment 9250 [details] Patch Changed the way the repaintRect is computed to account for the vertical edges of all lines that have been deleted, moved (before and after) or added. No test yet because I cannot get the tests to fail without the fix under the normal repaint test mechanism - the first repaint is big correct (or bigger than correct), for some reason (you can see this effect in Safari, too, i.e. in some instances the first time you mouseover and mouseout the repainting is correct, but from the second time onwards it isn't).
mitz
Comment 3 2006-07-08 11:08:43 PDT
Created attachment 9276 [details] Patch including change log and a repaint test
Dave Hyatt
Comment 4 2006-07-11 14:40:17 PDT
Comment on attachment 9276 [details] Patch including change log and a repaint test r=me
Alexey Proskuryakov
Comment 5 2006-07-24 09:49:23 PDT
Committed revision 15602.
Note You need to log in before you can comment on or make changes to this bug.