Chromium Code Reviews| Index: third_party/WebKit/Source/core/layout/LayoutTableSection.cpp |
| diff --git a/third_party/WebKit/Source/core/layout/LayoutTableSection.cpp b/third_party/WebKit/Source/core/layout/LayoutTableSection.cpp |
| index 25fb70ab90f774205c1b74dae700f9f6112bb61c..9745fe13110a2c64c41e799df9c003e9a23f3ecc 100644 |
| --- a/third_party/WebKit/Source/core/layout/LayoutTableSection.cpp |
| +++ b/third_party/WebKit/Source/core/layout/LayoutTableSection.cpp |
| @@ -1966,6 +1966,7 @@ int LayoutTableSection::logicalHeightForRow( |
| const LayoutTableRow& rowObject) const { |
| unsigned rowIndex = rowObject.rowIndex(); |
| int logicalHeight = 0; |
| + |
|
eae
2016/10/19 21:57:42
Could you please add an ASSERT that rowIndex < siz
mstensho (USE GERRIT)
2016/10/20 08:59:08
Aye aye, captain!
I have to learn to stop adding
|
| const Row& row = m_grid[rowIndex].row; |
| unsigned cols = row.size(); |
| for (unsigned colIndex = 0; colIndex < cols; colIndex++) { |
| @@ -1980,6 +1981,12 @@ int LayoutTableSection::logicalHeightForRow( |
| std::max(logicalHeight, cell->logicalHeightForRowSizing()); |
| } |
| } |
| + |
| + if (m_grid[rowIndex].logicalHeight.isSpecified()) { |
| + LayoutUnit specifiedLogicalHeight = |
| + minimumValueForLength(m_grid[rowIndex].logicalHeight, LayoutUnit()); |
| + logicalHeight = std::max(logicalHeight, specifiedLogicalHeight.toInt()); |
| + } |
| return logicalHeight; |
| } |