Index: Source/core/rendering/RenderTableSection.cpp |
diff --git a/Source/core/rendering/RenderTableSection.cpp b/Source/core/rendering/RenderTableSection.cpp |
index a2e973a17b4907da82d871aab1a54c05302d223b..702df82b39bf603d39fa8c02d10d56f04b2c1887 100644 |
--- a/Source/core/rendering/RenderTableSection.cpp |
+++ b/Source/core/rendering/RenderTableSection.cpp |
@@ -546,7 +546,16 @@ void RenderTableSection::distributeRowSpanHeightToRows(SpanningRenderTableCells& |
if (spanningRowsHeight.rowWithOnlySpanningCells) |
updateRowsHeightHavingOnlySpanningCells(cell, spanningRowsHeight); |
- if (!spanningRowsHeight.totalRowsHeight || spanningRowsHeight.spanningCellHeightIgnoringBorderSpacing <= spanningRowsHeight.totalRowsHeight) { |
+ |
+ if (!spanningRowsHeight.totalRowsHeight) { |
+ if (spanningRowsHeight.spanningCellHeightIgnoringBorderSpacing) |
Julien - ping for review
2013/11/13 07:13:17
Please add a comment as to why we do this, ideally
a.suchit
2013/11/19 11:59:56
Done.
|
+ m_rowPos[rowIndex + rowSpan] += spanningRowsHeight.spanningCellHeightIgnoringBorderSpacing + borderSpacingForRow(rowIndex + rowSpan -1); |
Julien - ping for review
2013/11/13 07:13:17
Coding style violation: there should be a space af
a.suchit
2013/11/19 11:59:56
Done.
|
+ |
+ extraHeightToPropagate = m_rowPos[rowIndex + rowSpan] - originalBeforePosition; |
+ continue; |
+ } |
+ |
+ if (spanningRowsHeight.spanningCellHeightIgnoringBorderSpacing <= spanningRowsHeight.totalRowsHeight) { |
extraHeightToPropagate = m_rowPos[rowIndex + rowSpan] - originalBeforePosition; |
continue; |
} |