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

third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp

 /*
  * Copyright (C) 2006 Oliver Hunt <ojh16@student.canterbury.ac.nz>
  * Copyright (C) 2006 Apple Computer Inc.
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2008 Rob Buis <buis@kde.org>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 179 matching lines @@
     int offset = closestDistanceBox->offsetForPositionInFragment(*closestDistanceFragment, LayoutUnit(absolutePoint.x() - closestDistancePosition), true);
     return createPositionWithAffinity(offset + closestDistanceBox->start(), offset > 0 ? VP_UPSTREAM_IF_POSSIBLE : TextAffinity::Downstream);
 }
 
 namespace {
 
 inline bool isValidSurrogatePair(const TextRun& run, unsigned index)
 {
     if (!U16_IS_LEAD(run[index]))
         return false;
-    if (index + 1 >= static_cast<unsigned>(run.length()))
+    if (index + 1 >= run.length())
         return false;
     return U16_IS_TRAIL(run[index + 1]);
 }
 
 TextRun constructTextRun(LayoutSVGInlineText& text, unsigned position, unsigned length, TextDirection textDirection)
 {
     const ComputedStyle& style = text.styleRef();
 
     TextRun run(static_cast<const LChar*>(nullptr) // characters, will be set below if non-zero.
         , 0 // length, will be set below if non-zero.
@@ skipping 57 matching lines @@
 } // namespace
 
 void LayoutSVGInlineText::addMetricsFromRun(
     const TextRun& run, bool& lastCharacterWasWhiteSpace)
 {
     Vector<CharacterRange> charRanges = scaledFont().individualCharacterRanges(run);
     synthesizeGraphemeWidths(run, charRanges);
 
     const float cachedFontHeight = scaledFont().getFontMetrics().floatHeight() / m_scalingFactor;
     const bool preserveWhiteSpace = styleRef().whiteSpace() == PRE;
-    const unsigned runLength = static_cast<unsigned>(run.length());
+    const unsigned runLength = run.length();
 
     // TODO(pdr): Character-based iteration is ambiguous and error-prone. It
     // should be unified under a single concept. See: https://crbug.com/593570
     unsigned characterIndex = 0;
     while (characterIndex < runLength) {
         bool currentCharacterIsWhiteSpace = run[characterIndex] == ' ';
         if (!preserveWhiteSpace && lastCharacterWasWhiteSpace && currentCharacterIsWhiteSpace) {
             m_metrics.append(SVGTextMetrics(SVGTextMetrics::SkippedSpaceMetrics));
             characterIndex++;
             continue;
@@ skipping 89 matching lines @@
 
 PassRefPtr<StringImpl> LayoutSVGInlineText::originalText() const
 {
     RefPtr<StringImpl> result = LayoutText::originalText();
     if (!result)
         return nullptr;
     return normalizeWhitespace(result);
 }
 
 } // namespace blink