RESOLVED FIXED 23487
Eliminate the borderTopExtra lie from RenderBox::y() 
https://bugs.webkit.org/show_bug.cgi?id=23487
Summary Eliminate the borderTopExtra lie from RenderBox::y()
Dave Hyatt
Reported 2009-01-22 17:23:58 PST
Rework table cells so that the y() coordinate never lies.
Attachments
Patch that not only passes but also progresses layout tests :) (56.17 KB, patch)
2009-01-22 18:41 PST, Dave Hyatt 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dave Hyatt
Comment 1 2009-01-22 18:41:04 PST
Created attachment 26950 [details] Patch that not only passes but also progresses layout tests :)
Oliver Hunt
Comment 2 2009-01-22 19:00:50 PST
Comment on attachment 26950 [details] Patch that not only passes but also progresses layout tests :) r=me, afaict this looks like it should help perf, but maybe i'm mistaken. anyhoo, r=me is conditional on perf being good. also looking at the bit fields for m_intrinsicPaddingTop, m_intrinsicPaddingBottom, and m_widthChanged -- i think we should look at rearranging these to place n_widthChanged first, to reduce the liklihood that we'll end up with either horrific load alignment and/or an unnecessary word being added to RenderTableCell
Dave Hyatt
Comment 3 2009-01-22 19:48:29 PST
Fixed.
Note You need to log in before you can comment on or make changes to this bug.