RESOLVED FIXED Bug 13873
Incomplete repaint of replaced element's box shadow 
https://bugs.webkit.org/show_bug.cgi?id=13873
Summary Incomplete repaint of replaced element's box shadow
mitz
Reported 2007-05-25 06:01:42 PDT
Replaced elements don't include box shadow in their (currently non-existent) visual overflow.
Attachments
Partial fix (2.21 KB, patch)
2007-05-25 06:08 PDT, mitz 		no flags
DetailsFormatted DiffDiff
Implement overflow for replaced elements (58.91 KB, patch)
2007-07-06 11:04 PDT, mitz 		hyatt: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2007-05-25 06:08:22 PDT
Created attachment 14720 [details] Partial fix This patch works for all cases except when the box shadow itself changes: because it doesn't cache the overflows it has no way of knowing the overflow before the change. I see three possible ways around this: 1) Use a "rare data" scheme for the overflows (there appear to be 13 unused bits in RenderReplaced). 2) Implement RenderReplaced::setStyle() that repaints when box-shadow changes. 3) Add the repaint in RenderObject::setStyle(), similar to how it repaints on outline size changes.
mitz
Comment 2 2007-05-25 06:09:40 PDT
Hyatt, thoughts?
mitz
Comment 3 2007-07-06 11:04:28 PDT
Created attachment 15419 [details] Implement overflow for replaced elements
Dave Hyatt
Comment 4 2007-07-10 13:56:41 PDT
Comment on attachment 15419 [details] Implement overflow for replaced elements r=me
Mark Rowe (bdash)
Comment 5 2007-07-13 05:02:46 PDT
Landed in r24256.
Note You need to log in before you can comment on or make changes to this bug.