Change TextRun's length() and charactersLength() to return an unsigned

third_party/WebKit/Source/core/paint/TextPainter.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/paint/TextPainter.h"
 
 #include "core/CSSPropertyNames.h"
 #include "core/frame/Settings.h"
 #include "core/layout/LayoutObject.h"
 #include "core/layout/LayoutTextCombine.h"
@@ skipping 36 matching lines @@
     if (emphasisMark.isNull()) {
         m_emphasisMarkOffset = 0;
     } else if (position == TextEmphasisPositionOver) {
         m_emphasisMarkOffset = -m_font.getFontMetrics().ascent() - m_font.emphasisMarkDescent(emphasisMark);
     } else {
         ASSERT(position == TextEmphasisPositionUnder);
         m_emphasisMarkOffset = m_font.getFontMetrics().descent() + m_font.emphasisMarkAscent(emphasisMark);
     }
 }
 
-void TextPainter::paint(int startOffset, int endOffset, int length, const Style& textStyle, TextBlobPtr* cachedTextBlob)
+void TextPainter::paint(unsigned startOffset, unsigned endOffset, unsigned length, const Style& textStyle, TextBlobPtr* cachedTextBlob)
 {
     GraphicsContextStateSaver stateSaver(m_graphicsContext, false);
     updateGraphicsContext(textStyle, stateSaver);
     if (m_combinedText) {
         m_graphicsContext.save();
         m_combinedText->transformToInlineCoordinates(m_graphicsContext, m_textBounds);
         paintInternal<PaintText>(startOffset, endOffset, length, cachedTextBlob);
         m_graphicsContext.restore();
     } else {
         paintInternal<PaintText>(startOffset, endOffset, length, cachedTextBlob);
@@ skipping 108 matching lines @@
 
         // Text shadows are disabled when printing. http://crbug.com/258321
         if (isPrinting)
             selectionStyle.shadow = 0;
     }
 
     return selectionStyle;
 }
 
 template <TextPainter::PaintInternalStep step>
-void TextPainter::paintInternalRun(TextRunPaintInfo& textRunPaintInfo, int from, int to)
+void TextPainter::paintInternalRun(TextRunPaintInfo& textRunPaintInfo, unsigned from, unsigned to)
 {
     ASSERT(from <= textRunPaintInfo.run.length());
     ASSERT(to <= textRunPaintInfo.run.length());
 
     textRunPaintInfo.from = from;
     textRunPaintInfo.to = to;
 
     if (step == PaintEmphasisMark) {
         m_graphicsContext.drawEmphasisMarks(m_font, textRunPaintInfo, m_emphasisMark,
             FloatPoint(m_textOrigin) + IntSize(0, m_emphasisMarkOffset));
     } else {
         ASSERT(step == PaintText);
         m_graphicsContext.drawText(m_font, textRunPaintInfo, FloatPoint(m_textOrigin));
     }
 }
 
 template <TextPainter::PaintInternalStep Step>
-void TextPainter::paintInternal(int startOffset, int endOffset, int truncationPoint, TextBlobPtr* cachedTextBlob)
+void TextPainter::paintInternal(unsigned startOffset, unsigned endOffset, unsigned truncationPoint, TextBlobPtr* cachedTextBlob)
 {
     TextRunPaintInfo textRunPaintInfo(m_run);
     textRunPaintInfo.bounds = FloatRect(m_textBounds);
     if (startOffset <= endOffset) {
         // FIXME: We should be able to use cachedTextBlob in more cases.
         textRunPaintInfo.cachedTextBlob = cachedTextBlob;
         paintInternalRun<Step>(textRunPaintInfo, startOffset, endOffset);
     } else {
         if (endOffset > 0)
             paintInternalRun<Step>(textRunPaintInfo, 0, endOffset);
         if (startOffset < truncationPoint)
             paintInternalRun<Step>(textRunPaintInfo, startOffset, truncationPoint);
     }
 }
 
 void TextPainter::paintEmphasisMarkForCombinedText()
 {
     ASSERT(m_combinedText);
     TextRun placeholderTextRun(&ideographicFullStopCharacter, 1);
     FloatPoint emphasisMarkTextOrigin(m_textBounds.x().toFloat(), m_textBounds.y().toFloat() + m_font.getFontMetrics().ascent() + m_emphasisMarkOffset);
     TextRunPaintInfo textRunPaintInfo(placeholderTextRun);
     textRunPaintInfo.bounds = FloatRect(m_textBounds);
     m_graphicsContext.concatCTM(rotation(m_textBounds, Clockwise));
     m_graphicsContext.drawEmphasisMarks(m_combinedText->originalFont(), textRunPaintInfo, m_emphasisMark, emphasisMarkTextOrigin);
     m_graphicsContext.concatCTM(rotation(m_textBounds, Counterclockwise));
 }
 
 } // namespace blink