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

Unified Diff: Source/core/layout/TableLayoutAlgorithmAuto.h

Issue 988443003: Don't add artifical 1 pixel width to empty tables (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated Created 5 years, 6 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: Source/core/layout/TableLayoutAlgorithmAuto.h
diff --git a/Source/core/layout/TableLayoutAlgorithmAuto.h b/Source/core/layout/TableLayoutAlgorithmAuto.h
index 22dc635d7b784cf27a7a7fe37dd5677045b2725a..afdae6d1d39d442532ef51bc33e1c22e4d2f86fa 100644
--- a/Source/core/layout/TableLayoutAlgorithmAuto.h
+++ b/Source/core/layout/TableLayoutAlgorithmAuto.h
@@ -33,7 +33,8 @@ class LayoutTableCell;
enum CellsToProcess {
AllCells,
- NonEmptyCells
+ NonEmptyCells,
+ EmptyCells
};
enum DistributionMode {
@@ -75,6 +76,7 @@ private:
, effectiveMaxLogicalWidth(0)
, computedLogicalWidth(0)
, emptyCellsOnly(true)
+ , columnHasNoCells(true)
{
}
@@ -86,6 +88,8 @@ private:
int effectiveMaxLogicalWidth;
int computedLogicalWidth;
bool emptyCellsOnly;
+ bool columnHasNoCells;
+ int clampedEffectiveMaxLogicalWidth() { return std::max<int>(1, effectiveMaxLogicalWidth); }
};
Vector<Layout, 4> m_layoutStruct;
« no previous file with comments | « LayoutTests/tables/mozilla/bugs/bug222336-expected.txt ('k') | Source/core/layout/TableLayoutAlgorithmAuto.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698