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

Unified Diff: Source/core/paint/InlineTextBoxPainter.cpp

Issue 890853002: Revert of text-combine should scale rather than fall back to none when wide (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 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: Source/core/paint/InlineTextBoxPainter.cpp
diff --git a/Source/core/paint/InlineTextBoxPainter.cpp b/Source/core/paint/InlineTextBoxPainter.cpp
index b2e2b71cf6b35d5dc0e65bb842e22adb24e6e0f8..5187bd0ff4db450040a6de175ead93cfda475994 100644
--- a/Source/core/paint/InlineTextBoxPainter.cpp
+++ b/Source/core/paint/InlineTextBoxPainter.cpp
@@ -107,21 +107,11 @@ void InlineTextBoxPainter::paint(const PaintInfo& paintInfo, const LayoutPoint&
boxOrigin.move(adjustedPaintOffset.x().toFloat(), adjustedPaintOffset.y().toFloat());
FloatRect boxRect(boxOrigin, FloatSize(m_inlineTextBox.logicalWidth(), m_inlineTextBox.logicalHeight()));
- bool shouldRotate = false;
- RenderCombineText* combinedText = nullptr;
- if (!m_inlineTextBox.isHorizontal()) {
- if (styleToUse->hasTextCombine() && m_inlineTextBox.renderer().isCombineText()) {
- combinedText = &toRenderCombineText(m_inlineTextBox.renderer());
- if (!combinedText->isCombined())
- combinedText = nullptr;
- }
- if (combinedText) {
- combinedText->updateFont();
- } else {
- shouldRotate = true;
- context->concatCTM(TextPainter::rotation(boxRect, TextPainter::Clockwise));
- }
- }
+ RenderCombineText* combinedText = styleToUse->hasTextCombine() && m_inlineTextBox.renderer().isCombineText() && toRenderCombineText(m_inlineTextBox.renderer()).isCombined() ? &toRenderCombineText(m_inlineTextBox.renderer()) : 0;
+
+ bool shouldRotate = !m_inlineTextBox.isHorizontal() && !combinedText;
+ if (shouldRotate)
+ context->concatCTM(TextPainter::rotation(boxRect, TextPainter::Clockwise));
// Determine whether or not we have composition underlines to draw.
bool containsComposition = m_inlineTextBox.renderer().node() && m_inlineTextBox.renderer().frame()->inputMethodController().compositionNode() == m_inlineTextBox.renderer().node();
« no previous file with comments | « LayoutTests/virtual/antialiasedtext/fast/text/font-variant-width-expected.txt ('k') | Source/core/paint/TextPainter.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698