Let RenderTable reach table cells needing overflow recalc

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 a5a90e380223bfb62aa2484c3ef5cabc7342914f..71a02d8c9873189f301453dbcd3b92dd44227067 100644
--- a/Source/core/rendering/RenderTable.cpp
+++ b/Source/core/rendering/RenderTable.cpp
@@ -393,6 +393,19 @@ void RenderTable::simplifiedNormalFlowLayout()
     }
 }
 
+bool RenderTable::recalcChildOverflowAfterStyleChange()
+{
+    ASSERT(childNeedsOverflowRecalcAfterStyleChange());
+    clearChildNeedsOverflowRecalcAfterStyleChange();
+
+    bool childrenOverflowChanged = false;
+    for (RenderTableSection* section = topSection(); section; section = sectionBelow(section)) {
+        if (section->childNeedsOverflowRecalcAfterStyleChange() && section->recalcChildOverflowAfterStyleChange())
+            childrenOverflowChanged = true;

[Julien - ping for review, 2014/11/25 18:33:10]

This is better written:

childrenOverflowChanged |= section->childNeedsOverflowRecalcAfterStyleChange()
&& section->recalcChildOverflowAfterStyleChange();

However here, you don't need to continue if it's true:

if (section->childNeedsOverflowRecalcAfterStyleChange() &&
section->recalcChildOverflowAfterStyleChange())
    return true;

[Xianzhu, 2014/11/25 20:17:12]

We still need to continue the loop to call recalcChildOverflowAfterStyleChange
for other sections.

It seems that embedding section->recalcChildOverflowAfterStyleChange() in a
complex boolean expression let us think we are just calculating the boolean
result and overlook the real action.

Would this be better:

        if (!section->childNeedsOverflowRecalcAfterStyleChange())
            continue;
        if (section->recalcChildOverflowAfterStyleChange())
            childrenOverflowChanged = true;

+    }
+    return childrenOverflowChanged;
+}
+
 void RenderTable::layout()
 {
     ASSERT(needsLayout());