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..55ac926b320090b88b536fdf717ceab19a8df9c2 100644
--- a/Source/core/rendering/RenderTable.cpp
+++ b/Source/core/rendering/RenderTable.cpp
@@ -387,6 +387,11 @@ void RenderTable::distributeExtraLogicalHeight(int extraLogicalHeight)
 
 void RenderTable::simplifiedNormalFlowLayout()
 {
+    // FIXME: We should walk through the items in the tree in tree order to do the layout here
+    // instead of walking through individual parts of the tree. crbug.com/442737
+    for (auto& caption : m_captions)
+        caption->layoutIfNeeded();

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

You're laying out the captions but nowhere are you placing them (see
RenderTable::layoutCaption()), which seems wrong. It's possible that under
normal flow layout, we don't expect the captions to change size (in which case,
we should ASSERT it).

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

Done.

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