| Index: Source/core/rendering/InlineTextBox.cpp
|
| diff --git a/Source/core/rendering/InlineTextBox.cpp b/Source/core/rendering/InlineTextBox.cpp
|
| index 91d7ad6af6cfa751f936b35ea7715cd7dd9b8674..b89f6708de9431053285e06f5d755bffb527bc7f 100644
|
| --- a/Source/core/rendering/InlineTextBox.cpp
|
| +++ b/Source/core/rendering/InlineTextBox.cpp
|
| @@ -206,12 +206,12 @@ LayoutRect InlineTextBox::localSelectionRect(int startPos, int endPos)
|
| bool respectHyphen = ePos == m_len && hasHyphen();
|
| TextRun textRun = constructTextRun(styleToUse, font, respectHyphen ? &charactersWithHyphen : 0);
|
|
|
| - FloatPoint startingPoint = FloatPoint(logicalLeft(), selTop.toFloat());
|
| + FloatPointWillBeLayoutPoint startingPoint = FloatPointWillBeLayoutPoint(logicalLeft(), selTop.toFloat());
|
| LayoutRect r;
|
| if (sPos || ePos != static_cast<int>(m_len))
|
| r = enclosingIntRect(font.selectionRectForText(textRun, startingPoint, selHeight, sPos, ePos));
|
| else // Avoid computing the font width when the entire line box is selected as an optimization.
|
| - r = enclosingIntRect(FloatRect(startingPoint, FloatSize(m_logicalWidth, selHeight.toFloat())));
|
| + r = enclosingIntRect(FloatRectWillBeLayoutRect(startingPoint, FloatSizeWillBeLayoutSize(m_logicalWidth, selHeight.toFloat())));
|
|
|
| LayoutUnit logicalWidth = r.width();
|
| if (r.x() > logicalRight())
|
| @@ -248,7 +248,7 @@ void InlineTextBox::attachLine()
|
| renderer().attachTextBox(this);
|
| }
|
|
|
| -float InlineTextBox::placeEllipsisBox(bool flowIsLTR, float visibleLeftEdge, float visibleRightEdge, float ellipsisWidth, float &truncatedWidth, bool& foundBox)
|
| +FloatWillBeLayoutUnit InlineTextBox::placeEllipsisBox(bool flowIsLTR, FloatWillBeLayoutUnit visibleLeftEdge, FloatWillBeLayoutUnit visibleRightEdge, FloatWillBeLayoutUnit ellipsisWidth, FloatWillBeLayoutUnit &truncatedWidth, bool& foundBox)
|
| {
|
| if (foundBox) {
|
| m_truncation = cFullTruncation;
|
| @@ -256,7 +256,7 @@ float InlineTextBox::placeEllipsisBox(bool flowIsLTR, float visibleLeftEdge, flo
|
| }
|
|
|
| // For LTR this is the left edge of the box, for RTL, the right edge in parent coordinates.
|
| - float ellipsisX = flowIsLTR ? visibleRightEdge - ellipsisWidth : visibleLeftEdge + ellipsisWidth;
|
| + FloatWillBeLayoutUnit ellipsisX = flowIsLTR ? visibleRightEdge - ellipsisWidth : visibleLeftEdge + ellipsisWidth;
|
|
|
| // Criteria for full truncation:
|
| // LTR: the left edge of the ellipsis is to the left of our text run.
|
| @@ -299,7 +299,7 @@ float InlineTextBox::placeEllipsisBox(bool flowIsLTR, float visibleLeftEdge, flo
|
|
|
| // If we got here that means that we were only partially truncated and we need to return the pixel offset at which
|
| // to place the ellipsis.
|
| - float widthOfVisibleText = renderer().width(m_start, offset, textPos(), flowIsLTR ? LTR : RTL, isFirstLineStyle());
|
| + FloatWillBeLayoutUnit widthOfVisibleText = renderer().width(m_start, offset, textPos(), flowIsLTR ? LTR : RTL, isFirstLineStyle());
|
|
|
| // The ellipsis needs to be placed just after the last visible character.
|
| // Where "after" is defined by the flow directionality, not the inline
|
| @@ -326,9 +326,9 @@ bool InlineTextBox::nodeAtPoint(const HitTestRequest& request, HitTestResult& re
|
| if (isLineBreak())
|
| return false;
|
|
|
| - FloatPoint boxOrigin = locationIncludingFlipping();
|
| + FloatPointWillBeLayoutPoint boxOrigin = locationIncludingFlipping();
|
| boxOrigin.moveBy(accumulatedOffset);
|
| - FloatRect rect(boxOrigin, size());
|
| + FloatRectWillBeLayoutRect rect(boxOrigin, size());
|
| if (m_truncation != cFullTruncation && visibleToHitTestRequest(request) && locationInContainer.intersects(rect)) {
|
| renderer().updateHitTestResult(result, flipForWritingMode(locationInContainer.point() - toLayoutSize(accumulatedOffset)));
|
| if (!result.addNodeToRectBasedTestResult(renderer().node(), request, locationInContainer, rect))
|
| @@ -383,12 +383,12 @@ void InlineTextBox::selectionStartEnd(int& sPos, int& ePos) const
|
| ePos = std::min(endPos - m_start, (int)m_len);
|
| }
|
|
|
| -void InlineTextBox::paintDocumentMarker(GraphicsContext* pt, const FloatPoint& boxOrigin, DocumentMarker* marker, RenderStyle* style, const Font& font, bool grammar)
|
| +void InlineTextBox::paintDocumentMarker(GraphicsContext* pt, const FloatPointWillBeLayoutPoint& boxOrigin, DocumentMarker* marker, RenderStyle* style, const Font& font, bool grammar)
|
| {
|
| InlineTextBoxPainter(*this).paintDocumentMarker(pt, boxOrigin, marker, style, font, grammar);
|
| }
|
|
|
| -void InlineTextBox::paintTextMatchMarker(GraphicsContext* pt, const FloatPoint& boxOrigin, DocumentMarker* marker, RenderStyle* style, const Font& font)
|
| +void InlineTextBox::paintTextMatchMarker(GraphicsContext* pt, const FloatPointWillBeLayoutPoint& boxOrigin, DocumentMarker* marker, RenderStyle* style, const Font& font)
|
| {
|
| InlineTextBoxPainter(*this).paintTextMatchMarker(pt, boxOrigin, marker, style, font);
|
| }
|
| @@ -403,7 +403,7 @@ int InlineTextBox::caretMaxOffset() const
|
| return m_start + m_len;
|
| }
|
|
|
| -float InlineTextBox::textPos() const
|
| +FloatWillBeLayoutUnit InlineTextBox::textPos() const
|
| {
|
| // When computing the width of a text run, RenderBlock::computeInlineDirectionPositionsForLine() doesn't include the actual offset
|
| // from the containing block edge in its measurement. textPos() should be consistent so the text are rendered in the same width.
|
| @@ -412,7 +412,7 @@ float InlineTextBox::textPos() const
|
| return logicalLeft() - root().logicalLeft();
|
| }
|
|
|
| -int InlineTextBox::offsetForPosition(float lineOffset, bool includePartialGlyphs) const
|
| +int InlineTextBox::offsetForPosition(FloatWillBeLayoutUnit lineOffset, bool includePartialGlyphs) const
|
| {
|
| if (isLineBreak())
|
| return 0;
|
| @@ -430,7 +430,7 @@ int InlineTextBox::offsetForPosition(float lineOffset, bool includePartialGlyphs
|
| return font.offsetForPosition(constructTextRun(style, font), lineOffset - logicalLeft(), includePartialGlyphs);
|
| }
|
|
|
| -float InlineTextBox::positionForOffset(int offset) const
|
| +FloatWillBeLayoutUnit InlineTextBox::positionForOffset(int offset) const
|
| {
|
| ASSERT(offset >= m_start);
|
| ASSERT(offset <= m_start + m_len);
|
| @@ -474,7 +474,7 @@ bool InlineTextBox::containsCaretOffset(int offset) const
|
| return true;
|
| }
|
|
|
| -void InlineTextBox::characterWidths(Vector<float>& widths) const
|
| +void InlineTextBox::characterWidths(Vector<FloatWillBeLayoutUnit>& widths) const
|
| {
|
| FontCachePurgePreventer fontCachePurgePreventer;
|
|
|
| @@ -484,7 +484,7 @@ void InlineTextBox::characterWidths(Vector<float>& widths) const
|
| TextRun textRun = constructTextRun(styleToUse, font);
|
|
|
| SimpleShaper shaper(&font, textRun);
|
| - float lastWidth = 0;
|
| + FloatWillBeLayoutUnit lastWidth = 0;
|
| widths.resize(m_len);
|
| for (unsigned i = 0; i < m_len; i++) {
|
| shaper.advance(i + 1);
|
|
|