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

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

Issue 630093005: Move paint code from RootInlineBox and EllipsisBox to RootInlineBoxPainter and EllipsisBoxPainter. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 2 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 | « Source/core/rendering/EllipsisBox.h ('k') | Source/core/rendering/RootInlineBox.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/EllipsisBox.cpp
diff --git a/Source/core/rendering/EllipsisBox.cpp b/Source/core/rendering/EllipsisBox.cpp
index 5c4dd6c1efc1334890b24dca3968da5deefec4fc..bbdba5f34b816631b46a92f79ffdfa4060b2da68 100644
--- a/Source/core/rendering/EllipsisBox.cpp
+++ b/Source/core/rendering/EllipsisBox.cpp
@@ -20,52 +20,22 @@
#include "config.h"
#include "core/rendering/EllipsisBox.h"
+#include "core/paint/EllipsisBoxPainter.h"
#include "core/rendering/HitTestResult.h"
#include "core/rendering/InlineTextBox.h"
#include "core/rendering/PaintInfo.h"
#include "core/rendering/RenderBlock.h"
#include "core/rendering/RootInlineBox.h"
-#include "core/rendering/TextPainter.h"
#include "core/rendering/TextRunConstructor.h"
#include "core/rendering/style/ShadowList.h"
#include "platform/fonts/Font.h"
-#include "platform/graphics/GraphicsContextStateSaver.h"
#include "platform/text/TextRun.h"
namespace blink {
void EllipsisBox::paint(PaintInfo& paintInfo, const LayoutPoint& paintOffset, LayoutUnit lineTop, LayoutUnit lineBottom)
{
- GraphicsContext* context = paintInfo.context;
- RenderStyle* style = renderer().style(isFirstLineStyle());
- const Font& font = style->font();
- FloatPoint boxOrigin = locationIncludingFlipping();
- boxOrigin.moveBy(FloatPoint(paintOffset));
- if (!isHorizontal())
- boxOrigin.move(0, -virtualLogicalHeight());
- FloatRect boxRect(boxOrigin, LayoutSize(logicalWidth(), virtualLogicalHeight()));
- GraphicsContextStateSaver stateSaver(*context);
- if (!isHorizontal())
- context->concatCTM(InlineTextBox::rotation(boxRect, InlineTextBox::Clockwise));
- FloatPoint textOrigin(boxOrigin.x(), boxOrigin.y() + font.fontMetrics().ascent());
-
- bool isPrinting = renderer().document().printing();
- bool haveSelection = !isPrinting && paintInfo.phase != PaintPhaseTextClip && selectionState() != RenderObject::SelectionNone;
-
- if (haveSelection)
- paintSelection(context, boxOrigin, style, font);
- else if (paintInfo.phase == PaintPhaseSelection)
- return;
-
- TextPainter::Style textStyle = TextPainter::textPaintingStyle(renderer(), style, paintInfo.forceBlackText(), isPrinting);
- if (haveSelection)
- textStyle = TextPainter::selectionPaintingStyle(renderer(), true, paintInfo.forceBlackText(), isPrinting, textStyle);
-
- TextRun textRun = constructTextRun(&renderer(), font, m_str, style, TextRun::AllowTrailingExpansion);
- TextPainter textPainter(context, font, textRun, textOrigin, boxRect, isHorizontal());
- textPainter.paint(0, m_str.length(), m_str.length(), textStyle);
-
- paintMarkupBox(paintInfo, paintOffset, lineTop, lineBottom, style);
+ EllipsisBoxPainter(*this).paint(paintInfo, paintOffset, lineTop, lineBottom);
}
InlineBox* EllipsisBox::markupBox() const
@@ -87,18 +57,6 @@ InlineBox* EllipsisBox::markupBox() const
return anchorBox;
}
-void EllipsisBox::paintMarkupBox(PaintInfo& paintInfo, const LayoutPoint& paintOffset, LayoutUnit lineTop, LayoutUnit lineBottom, RenderStyle* style)
-{
- InlineBox* markupBox = this->markupBox();
- if (!markupBox)
- return;
-
- LayoutPoint adjustedPaintOffset = paintOffset;
- adjustedPaintOffset.move(x() + m_logicalWidth - markupBox->x(),
- y() + style->fontMetrics().ascent() - (markupBox->y() + markupBox->renderer().style(isFirstLineStyle())->fontMetrics().ascent()));
- markupBox->paint(paintInfo, adjustedPaintOffset, lineTop, lineBottom);
-}
-
IntRect EllipsisBox::selectionRect()
{
RenderStyle* style = renderer().style(isFirstLineStyle());
@@ -106,29 +64,6 @@ IntRect EllipsisBox::selectionRect()
return enclosingIntRect(font.selectionRectForText(constructTextRun(&renderer(), font, m_str, style, TextRun::AllowTrailingExpansion), IntPoint(logicalLeft(), logicalTop() + root().selectionTopAdjustedForPrecedingBlock()), root().selectionHeightAdjustedForPrecedingBlock()));
}
-void EllipsisBox::paintSelection(GraphicsContext* context, const FloatPoint& boxOrigin, RenderStyle* style, const Font& font)
-{
- Color textColor = renderer().resolveColor(style, CSSPropertyColor);
- Color c = renderer().selectionBackgroundColor();
- if (!c.alpha())
- return;
-
- // If the text color ends up being the same as the selection background, invert the selection
- // background.
- if (textColor == c)
- c = Color(0xff - c.red(), 0xff - c.green(), 0xff - c.blue());
-
- GraphicsContextStateSaver stateSaver(*context);
- LayoutUnit selectionBottom = root().selectionBottom();
- LayoutUnit top = root().selectionTop();
- LayoutUnit h = root().selectionHeight();
- const int deltaY = roundToInt(renderer().style()->isFlippedLinesWritingMode() ? selectionBottom - logicalBottom() : logicalTop() - top);
- const FloatPoint localOrigin(boxOrigin.x(), boxOrigin.y() - deltaY);
- FloatRect clipRect(localOrigin, FloatSize(m_logicalWidth, h.toFloat()));
- context->clip(clipRect);
- context->drawHighlightForText(font, constructTextRun(&renderer(), font, m_str, style, TextRun::AllowTrailingExpansion), localOrigin, h, c);
-}
-
bool EllipsisBox::nodeAtPoint(const HitTestRequest& request, HitTestResult& result, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset, LayoutUnit lineTop, LayoutUnit lineBottom)
{
LayoutPoint adjustedLocation = accumulatedOffset + roundedLayoutPoint(topLeft());
« no previous file with comments | « Source/core/rendering/EllipsisBox.h ('k') | Source/core/rendering/RootInlineBox.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698