Layout captions in simplifiedNormalFlowLayout for tables.

Source/core/rendering/RenderTable.cpp

Index: Source/core/rendering/RenderTable.cpp
diff --git a/Source/core/rendering/RenderTable.cpp b/Source/core/rendering/RenderTable.cpp
index 81d33641eeeffb0f9723f0eb3509c4dbc3457edd..7454e4de90cb72261a8c3773199c40ad8e362f1b 100644
--- a/Source/core/rendering/RenderTable.cpp
+++ b/Source/core/rendering/RenderTable.cpp
@@ -387,6 +387,9 @@ void RenderTable::distributeExtraLogicalHeight(int extraLogicalHeight)
 
 void RenderTable::simplifiedNormalFlowLayout()
 {
+    for (auto& caption : m_captions)

[mstensho (USE GERRIT), 2014/12/16 15:27:01]

It may not matter much (since I assume and really hope that the children's
position remain untouched during simplified layout), but why not lay out the
table children in tree order (regardless of being captions or header/body/footer
sections), like in RenderTable::layout()?

[dsinclair, 2014/12/16 15:40:32]

Filed crbug.com/442737 and added a comment. I'd like to keep this CL about
fixing the crash, we can do the refactoring later to make it nicer. That sound
good?

[mstensho (USE GERRIT), 2014/12/16 15:43:31]

Acknowledged.

[Julien - ping for review, 2014/12/16 16:05:53]

Captions needs to be laid out first so that we can stack them on top of each
other (at least for those that are at the top) so I am not sure this is such a
great idea.

[dsinclair, 2014/12/16 17:23:18]

So, you think this should be removed? Or should we discuss it in the bug and
remove later if desired?

[mstensho (USE GERRIT), 2014/12/16 20:37:07]

Isn't it normally the parent that positions the child? In the simplified layout
case, I was assuming that no repositioning was going to take place. After all,
nobody has been bothering to reposition the sections (the code right below).
Same in RenderBlock::simplifiedNormalFlowLayout() - no repositioning.

I think the current proposed change is fine, but I suggested that they be laid
out in the same order as in layout(), just for the sake of consistency. Not that
I really think it would matter, behavior-wise. Considering that for a separate
CL is probably the right thing, though.

+        caption->layoutIfNeeded();
+
     for (RenderTableSection* section = topSection(); section; section = sectionBelow(section)) {
         section->layoutIfNeeded();
         section->computeOverflowFromCells();