Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(527)

Unified Diff: third_party/WebKit/Source/core/layout/LayoutTableSection.cpp

Issue 1396493005: Table rowspan cell's height resize when percent element present in the cell. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Layout test expectation changed. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 baae67a67e1add2ff80ab2d4d35a5e2e8975557f..6ec209bde16ad8c655909a5a4c460628a0a65f82 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTableSection.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutTableSection.cpp
@@ -790,6 +790,12 @@ int LayoutTableSection::calcRowLogicalHeight()
rowSpanCells.append(cell);
lastRowSpanCell = cell;
+ if (cell->hasOverrideLogicalContentHeight()) {
mstensho (USE GERRIT) 2016/01/12 10:06:41 Yeah, I suppose this was missing for rowspanned ce
a.suchit2 2016/01/12 14:37:02 Done.
+ cell->clearIntrinsicPadding();
+ cell->clearOverrideSize();
+ cell->forceChildLayout();
+ }
+
// Find out the baseline. The baseline is set on the first row in a rowSpan.
updateBaselineForCell(cell, r, baselineDescent);
}

Powered by Google App Engine
This is Rietveld 408576698