Revert 99212 - Stop abusing RenderTableSection::needsRecalcCells logic

Source/WebCore/rendering/RenderTableCell.cpp

 /*
  * Copyright (C) 1997 Martin Jones (mjones@kde.org)
  *           (C) 1997 Torben Weis (weis@kde.org)
  *           (C) 1998 Waldo Bastian (bastian@kde.org)
  *           (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 292 matching lines @@
 {
     // <http://www.w3.org/TR/2007/CR-CSS21-20070719/tables.html#height-layout>: The baseline of a cell is the baseline of
     // the first in-flow line box in the cell, or the first in-flow table-row in the cell, whichever comes first. If there
     // is no such line box or table-row, the baseline is the bottom of content edge of the cell box.
     LayoutUnit firstLineBaseline = firstLineBoxBaseline();
     if (firstLineBaseline != -1)
         return firstLineBaseline;
     return paddingBefore() + borderBefore() + contentLogicalHeight();
 }
 
+void RenderTableCell::styleWillChange(StyleDifference diff, const RenderStyle* newStyle)
+{
+    if (parent() && section() && style() && style()->height() != newStyle->height())
+        section()->setNeedsCellRecalc();
+
+    ASSERT(newStyle->display() == TABLE_CELL);
+
+    RenderBlock::styleWillChange(diff, newStyle);
+}
+
 void RenderTableCell::styleDidChange(StyleDifference diff, const RenderStyle* oldStyle)
 {
-    ASSERT(style()->display() == TABLE_CELL);
-
     RenderBlock::styleDidChange(diff, oldStyle);
     setHasBoxDecorations(true);
 
-    if (parent() && section() && oldStyle && style()->height() != oldStyle->height() && rowWasSet())
-        section()->rowLogicalHeightChanged(row());
-
     // If border was changed, notify table.
     if (parent()) {
         RenderTable* table = this->table();
         if (table && !table->selfNeedsLayout() && !table->normalChildNeedsLayout()&& oldStyle && oldStyle->border() != style()->border())
             table->invalidateCollapsedBorders();
     }
 }
 
 // The following rules apply for resolving conflicts and figuring out which border
 // to use.
@@ skipping 719 matching lines @@
         totalHeight -= scrollbarHeight;
         LayoutUnit newBeforePadding = (totalHeight - heightWithoutIntrinsicPadding) / 2;
         LayoutUnit newAfterPadding = totalHeight - heightWithoutIntrinsicPadding - newBeforePadding;
         setIntrinsicPaddingBefore(newBeforePadding);
         setIntrinsicPaddingAfter(newAfterPadding);
     } else
         setIntrinsicPaddingAfter(intrinsicPaddingAfter() - scrollbarHeight);
 }
 
 } // namespace WebCore