CSS specifies that the baseline of an inline table is the baseline of the first row. Previously, inline tables were floating off into nowhere. See http://www.w3.org/TR/CSS2/visudet.html#leading: "The baseline of an 'inline-table' is the baseline of the first row of the table."
Created attachment 32388 [details] patch I'm not sure that the patch fixes this in the right way. The layout test included certainly demonstrates the issue and the patch fixes that, but it seems to cause some layout test failures which I'm still looking at.
This is hyatt's area. Possibly Adele too.
Comment on attachment 32388 [details] patch Haven't heard much from the table folks. But this looks sane to me. Please fix your style error: +int RenderTable::baselinePosition(bool firstLine, bool isRootLineBox) const { + if (firstBody() && style()->display() == INLINE_TABLE) Make sure you've run the layout tests when landing.
Will land.
Comment on attachment 32388 [details] patch This patch fails the following tests: editing/pasteboard/innerText-inline-table.html editing/selection/iframe.html editing/selection/inline-table.html editing/selection/table-caret-3.html fast/css-generated-content/inline-display-types.html fast/encoding/utf-16-big-endian.html fast/encoding/utf-16-little-endian.html fast/inline-block/001.html fast/invalid/residual-style.html fast/repaint/float-overflow-right.html fast/repaint/float-overflow.html http/tests/misc/redirect-to-about-blank.html tables/mozilla/bugs/bug2479-2.html
Unassigning. This is no longer actionable by me.
Sorry, I didn't intend for anyone to land this patch just yet. It does 'fail' tests, but I believe that the baselines are currently wrong for those tests. I don't have time to look at this now, so I'll leave the patch r-'ed.
This was solved by Robert as part of bug 84167. Duplicating this bug as the other bug had a fix that landed. I believe Robert's coverage was enough to cover the test in this patch. *** This bug has been marked as a duplicate of bug 84167 ***