RenderTableSection Blink merge asserting
authorossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Nov 2013 17:40:59 +0000 (17:40 +0000)
committerossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Nov 2013 17:40:59 +0000 (17:40 +0000)
https://bugs.webkit.org/show_bug.cgi?id=124857

Patch by Laszlo Vidacs <lac@inf.u-szeged.hu> on 2013-11-28
Reviewed by Csaba Osztrogonác.

Source/WebCore:

Use border spacing at the end of all sections.

* rendering/RenderTableSection.cpp:
(WebCore::RenderTableSection::calcRowLogicalHeight):

LayoutTests:

Skipped tests enabled.

* TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@159848 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/TestExpectations
Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderTableSection.cpp

index 71e91bc..f1f6eae 100644 (file)
@@ -1,3 +1,14 @@
+2013-11-28  Laszlo Vidacs  <lac@inf.u-szeged.hu>
+
+        RenderTableSection Blink merge asserting
+        https://bugs.webkit.org/show_bug.cgi?id=124857
+
+        Reviewed by Csaba Osztrogonác.
+
+        Skipped tests enabled.
+
+        * TestExpectations:
+
 2013-11-28  Michał Pakuła vel Rutka  <m.pakula@samsung.com>
 
         Unreviewed EFL gardening
index 6ce4f99..c72896b 100644 (file)
@@ -69,7 +69,3 @@ fast/mediastream/MediaStream-clone.html [ Skip ]
 # The spec is not clear if the MediaStream ended event should be fired if stop is called on each MediaStream's track
 # Skipping it for now, then put it again when the spec decides it
 fast/mediastream/MediaStream-onended.html [ Skip ]
-
-# Recent Blink merge has caused these tests to crash
-webkit.org/b/124857 tables/mozilla/bugs/bug46268-3.html [ Skip ]
-webkit.org/b/124857 tables/mozilla_expected_failures/bugs/bug46268-4.html [ Skip ]
index 5ceec00..55858da 100644 (file)
@@ -1,3 +1,15 @@
+2013-11-28  Laszlo Vidacs  <lac@inf.u-szeged.hu>
+
+        RenderTableSection Blink merge asserting
+        https://bugs.webkit.org/show_bug.cgi?id=124857
+
+        Reviewed by Csaba Osztrogonác.
+
+        Use border spacing at the end of all sections.
+
+        * rendering/RenderTableSection.cpp:
+        (WebCore::RenderTableSection::calcRowLogicalHeight):
+
 2013-11-28  Antti Koivisto  <antti@apple.com>
 
         Remove feature: CSS variables
index df39b89..31ec20b 100644 (file)
@@ -366,6 +366,8 @@ int RenderTableSection::calcRowLogicalHeight()
         }
 
         // Add the border-spacing to our final position.
+        // Use table border-spacing even in non-top sections
+        spacing = table()->vBorderSpacing();
         m_rowPos[r + 1] += m_grid[r].rowRenderer ? spacing : 0;
         m_rowPos[r + 1] = std::max(m_rowPos[r + 1], m_rowPos[r]);
     }