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

Unified Diff: Source/core/rendering/FastTextAutosizer.cpp

Issue 218863002: [FastTextAutosizer] Fix the inline overlap issue. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: update expectations Created 6 years, 9 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
« no previous file with comments | « LayoutTests/fast/text-autosizing/table-inline-width-expected.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/FastTextAutosizer.cpp
diff --git a/Source/core/rendering/FastTextAutosizer.cpp b/Source/core/rendering/FastTextAutosizer.cpp
index 2297eab82cfa99a17ede246e92ac51458e06cee2..06d0c4a41867f629ef37baa60c25cba655d7ab0b 100644
--- a/Source/core/rendering/FastTextAutosizer.cpp
+++ b/Source/core/rendering/FastTextAutosizer.cpp
@@ -412,6 +412,9 @@ void FastTextAutosizer::inflate(RenderBlock* block)
multiplier = cluster->m_autosize ? clusterMultiplier(cluster) : 1.0f;
applyMultiplier(descendant, multiplier);
applyMultiplier(descendant->parent(), multiplier); // Parent handles line spacing.
+ // FIXME: Investigate why MarkOnlyThis is sufficient.
+ if (descendant->parent()->isRenderInline())
+ descendant->setPreferredLogicalWidthsDirty(MarkOnlyThis);
}
descendant = descendant->nextInPreOrder(block);
}
« no previous file with comments | « LayoutTests/fast/text-autosizing/table-inline-width-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698