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

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

Issue 1887933002: (CANCELED) Fix TextAutosizer not to scheduleRelayout() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: simplifiedLayout Created 4 years, 8 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/TableLayoutAlgorithmAuto.cpp
diff --git a/third_party/WebKit/Source/core/layout/TableLayoutAlgorithmAuto.cpp b/third_party/WebKit/Source/core/layout/TableLayoutAlgorithmAuto.cpp
index 0ff2aad54ee239d5a6080cad371f6bc07eecb043..69417088ddadff0aeb81773e365b1290eaf82eae 100644
--- a/third_party/WebKit/Source/core/layout/TableLayoutAlgorithmAuto.cpp
+++ b/third_party/WebKit/Source/core/layout/TableLayoutAlgorithmAuto.cpp
@@ -208,7 +208,9 @@ static bool shouldScaleColumns(LayoutTable* table)
void TableLayoutAlgorithmAuto::computeIntrinsicLogicalWidths(LayoutUnit& minWidth, LayoutUnit& maxWidth)
{
- TextAutosizer::TableLayoutScope textAutosizerTableLayoutScope(m_table);
+ // TODO(kojii): We should give a SubtreeLayoutScope to TextAutosizer if in
+ // layout, but it's hard to get it here.
cbiesinger 2016/04/15 17:08:08 Hmm yeah that one is tricky...
kojii 2016/04/15 17:58:44 Yeah...or, what we really need is to avoid schedul
+ TextAutosizer::TableLayoutScope textAutosizerTableLayoutScope(m_table, nullptr);
fullRecalc();

Powered by Google App Engine
This is Rietveld 408576698