The start and end arguments should be unsigned long in setSelectionRange() function

Source/core/html/HTMLTextFormControlElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  *
  * 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 153 matching lines @@
         updatePlaceholderText();
     HTMLElement* placeholder = placeholderElement();
     if (!placeholder)
         return;
 
     placeholder->setInlineStyleProperty(CSSPropertyDisplay, placeholderShouldBeVisible() ? CSSValueBlock : CSSValueNone);
 }
 
 void HTMLTextFormControlElement::setSelectionStart(int start)
 {
-    setSelectionRange(start, std::max(start, selectionEnd()), selectionDirection());
+    setSelectionRange(start, std::max<unsigned>(start, selectionEnd()), selectionDirection());
 }
 
 void HTMLTextFormControlElement::setSelectionEnd(int end)
 {
-    setSelectionRange(std::min(end, selectionStart()), end, selectionDirection());
+    setSelectionRange(std::min<unsigned>(end, selectionStart()), end, selectionDirection());
 }
 
 void HTMLTextFormControlElement::setSelectionDirection(const String& direction)
 {
     setSelectionRange(selectionStart(), selectionEnd(), direction);
 }
 
 void HTMLTextFormControlElement::select(NeedToDispatchSelectEvent eventBehaviour)
 {
     document().updateLayoutIgnorePendingStylesheets();
@@ skipping 69 matching lines @@
 
         if (newSelectionEnd > end)
             newSelectionEnd += delta;
         else if (newSelectionEnd > start)
             newSelectionEnd = start + replacementLength;
     }
 
     setSelectionRange(newSelectionStart, newSelectionEnd, SelectionHasNoDirection);
 }
 
-void HTMLTextFormControlElement::setSelectionRange(int start, int end, const String& directionString)
+void HTMLTextFormControlElement::setSelectionRange(unsigned start, unsigned end, const String& directionString)
 {
     TextFieldSelectionDirection direction = SelectionHasNoDirection;
     if (directionString == "forward")
         direction = SelectionHasForwardDirection;
     else if (directionString == "backward")
         direction = SelectionHasBackwardDirection;
 
     if (direction == SelectionHasNoDirection && document().frame() && document().frame()->editor().behavior().shouldConsiderSelectionAsDirectional())
         direction = SelectionHasForwardDirection;
 
     return setSelectionRange(start, end, direction);
 }
 
-static Position positionForIndex(HTMLElement* innerEditor, int index)
+static Position positionForIndex(HTMLElement* innerEditor, unsigned index)
 {
-    ASSERT(index >= 0);
     if (index == 0) {
         Node* node = NodeTraversal::next(*innerEditor, innerEditor);
         if (node && node->isTextNode())
             return Position(node, 0);
         return Position(innerEditor, 0);
     }
-    int remainingCharactersToMoveForward = index;
+    unsigned remainingCharactersToMoveForward = index;
     Node* lastBrOrText = innerEditor;
     for (Node& node : NodeTraversal::descendantsOf(*innerEditor)) {
         ASSERT(remainingCharactersToMoveForward >= 0);
         if (node.hasTagName(brTag)) {
             if (remainingCharactersToMoveForward == 0)
                 return positionBeforeNode(&node);
             --remainingCharactersToMoveForward;
             lastBrOrText = &node;
             continue;
         }
 
         if (node.isTextNode()) {
             Text& text = toText(node);
-            if (remainingCharactersToMoveForward < static_cast<int>(text.length()))
+            if (remainingCharactersToMoveForward < text.length())
                 return Position(&text, remainingCharactersToMoveForward);
             remainingCharactersToMoveForward -= text.length();
             lastBrOrText = &node;
             continue;
         }
 
         ASSERT_NOT_REACHED();
     }
     return lastPositionInOrAfterNode(lastBrOrText);
 }
 
-static int indexForPosition(HTMLElement* innerEditor, const Position& passedPosition)
+static unsigned indexForPosition(HTMLElement* innerEditor, const Position& passedPosition)
 {
     if (!innerEditor || !innerEditor->contains(passedPosition.anchorNode()) || passedPosition.isNull())
         return 0;
 
     if (positionBeforeNode(innerEditor) == passedPosition)
         return 0;
 
-    int index = 0;
+    unsigned index = 0;
     Node* startNode = passedPosition.computeNodeBeforePosition();
     if (!startNode)
         startNode = passedPosition.computeContainerNode();
     ASSERT(startNode);
     ASSERT(innerEditor->contains(startNode));
 
     for (Node* node = startNode; node; node = NodeTraversal::previous(*node, innerEditor)) {
         if (node->isTextNode()) {
-            int length = toText(*node).length();
+            unsigned length = toText(*node).length();
             if (node == passedPosition.computeContainerNode())
-                index += std::min(length, passedPosition.offsetInContainerNode());
+                index += std::min<unsigned>(length, passedPosition.offsetInContainerNode());

[philipj_slow, 2015/08/14 07:45:52]

offsetInContainerNode() returns int. Maybe assign it to a local variable and
assert that it is >= 0?

             else
                 index += length;
         } else if (node->hasTagName(brTag)) {
             ++index;
         }
     }
 
-    ASSERT(index >= 0);
     return index;
 }
 
-void HTMLTextFormControlElement::setSelectionRange(int start, int end, TextFieldSelectionDirection direction, NeedToDispatchSelectEvent eventBehaviour, SelectionOption selectionOption)
+void HTMLTextFormControlElement::setSelectionRange(unsigned start, unsigned end, TextFieldSelectionDirection direction, NeedToDispatchSelectEvent eventBehaviour, SelectionOption selectionOption)
 {
     if (openShadowRoot() || !isTextFormControl() || !inDocument())
         return;
 
-    const int editorValueLength = static_cast<int>(innerEditorValue().length());
-    ASSERT(editorValueLength >= 0);
-    end = std::max(std::min(end, editorValueLength), 0);
-    start = std::min(std::max(start, 0), end);
+    const unsigned editorValueLength = static_cast<unsigned>(innerEditorValue().length());

[philipj_slow, 2015/08/14 07:45:52]

String::length() is already unsigned.

[tanay.c, 2015/08/17 12:06:32]

Done.

+    end = std::min<unsigned>(end, editorValueLength);
+    start = std::min<unsigned>(start, end);
     cacheSelection(start, end, direction);
 
     if (selectionOption == NotChangeSelection || (selectionOption == ChangeSelectionIfFocused && document().focusedElement() != this)) {
         if (eventBehaviour == DispatchSelectEvent)
             scheduleSelectEvent();
         return;
     }
 
     LocalFrame* frame = document().frame();
     HTMLElement* innerEditor = innerEditorElement();
     if (!frame || !innerEditor)
         return;
 
     Position startPosition = positionForIndex(innerEditor, start);
     Position endPosition = start == end ? startPosition : positionForIndex(innerEditor, end);
 
-    ASSERT(start == indexForPosition(innerEditor, startPosition));
-    ASSERT(end == indexForPosition(innerEditor, endPosition));
+    ASSERT(start == (unsigned)indexForPosition(innerEditor, startPosition));

[philipj_slow, 2015/08/14 07:45:52]

Don't use C-style casts.

[tanay.c, 2015/08/17 12:06:32]

Done.

+    ASSERT(end == (unsigned)indexForPosition(innerEditor, endPosition));
 
     // startPosition and endPosition can be null position for example when
     // "-webkit-user-select: none" style attribute is specified.
     if (startPosition.isNotNull() && endPosition.isNotNull()) {
         ASSERT(startPosition.anchorNode()->shadowHost() == this
             && endPosition.anchorNode()->shadowHost() == this);
     }
     VisibleSelection newSelection;
     if (direction == SelectionHasBackwardDirection)
         newSelection.setWithoutValidation(endPosition, startPosition);
@@ skipping 24 matching lines @@
     Position indexPosition = pos.deepEquivalent().parentAnchoredEquivalent();
     if (enclosingTextFormControl(indexPosition) != this)
         return 0;
     ASSERT(indexPosition.document());
     RefPtrWillBeRawPtr<Range> range = Range::create(*indexPosition.document());
     range->setStart(innerEditorElement(), 0, ASSERT_NO_EXCEPTION);
     range->setEnd(indexPosition.computeContainerNode(), indexPosition.offsetInContainerNode(), ASSERT_NO_EXCEPTION);
     return TextIterator::rangeLength(range->startPosition(), range->endPosition());
 }
 
-int HTMLTextFormControlElement::selectionStart() const
+unsigned HTMLTextFormControlElement::selectionStart() const
 {
     if (!isTextFormControl())
         return 0;
     if (document().focusedElement() != this)
         return m_cachedSelectionStart;
 
     return computeSelectionStart();
 }
 
-int HTMLTextFormControlElement::computeSelectionStart() const
+unsigned HTMLTextFormControlElement::computeSelectionStart() const
 {
     ASSERT(isTextFormControl());
     LocalFrame* frame = document().frame();
     if (!frame)
         return 0;
 
     return indexForPosition(innerEditorElement(), frame->selection().start());
 }
 
-int HTMLTextFormControlElement::selectionEnd() const
+unsigned HTMLTextFormControlElement::selectionEnd() const
 {
     if (!isTextFormControl())
         return 0;
     if (document().focusedElement() != this)
         return m_cachedSelectionEnd;
     return computeSelectionEnd();
 }
 
-int HTMLTextFormControlElement::computeSelectionEnd() const
+unsigned HTMLTextFormControlElement::computeSelectionEnd() const
 {
     ASSERT(isTextFormControl());
     LocalFrame* frame = document().frame();
     if (!frame)
         return 0;
 
     return indexForPosition(innerEditorElement(), frame->selection().end());
 }
 
 static const AtomicString& directionString(TextFieldSelectionDirection direction)
@@ skipping 29 matching lines @@
 {
     ASSERT(isTextFormControl());
     LocalFrame* frame = document().frame();
     if (!frame)
         return SelectionHasNoDirection;
 
     const VisibleSelection& selection = frame->selection().selection();
     return selection.isDirectional() ? (selection.isBaseFirst() ? SelectionHasForwardDirection : SelectionHasBackwardDirection) : SelectionHasNoDirection;
 }
 
-static inline void setContainerAndOffsetForRange(Node* node, int offset, Node*& containerNode, int& offsetInContainer)
+static inline void setContainerAndOffsetForRange(Node* node, unsigned offset, Node*& containerNode, unsigned& offsetInContainer)
 {
     if (node->isTextNode()) {
         containerNode = node;
         offsetInContainer = offset;
     } else {
         containerNode = node->parentNode();
         offsetInContainer = node->nodeIndex() + offset;
     }
 }
 
 PassRefPtrWillBeRawPtr<Range> HTMLTextFormControlElement::selection() const
 {
     if (!layoutObject() || !isTextFormControl())
         return nullptr;
 
-    int start = m_cachedSelectionStart;
-    int end = m_cachedSelectionEnd;
+    unsigned start = m_cachedSelectionStart;
+    unsigned end = m_cachedSelectionEnd;
 
     ASSERT(start <= end);
     HTMLElement* innerText = innerEditorElement();
     if (!innerText)
         return nullptr;
 
     if (!innerText->hasChildren())
         return Range::create(document(), innerText, 0, innerText, 0);
 
-    int offset = 0;
+    unsigned offset = 0;
     Node* startNode = 0;
     Node* endNode = 0;
     for (Node& node : NodeTraversal::descendantsOf(*innerText)) {
         ASSERT(!node.hasChildren());
         ASSERT(node.isTextNode() || isHTMLBRElement(node));
         int length = node.isTextNode() ? lastOffsetInNode(&node) : 1;
 
         if (offset <= start && start <= offset + length)
             setContainerAndOffsetForRange(&node, start - offset, startNode, start);
 
@@ skipping 466 matching lines @@
 }
 
 void HTMLTextFormControlElement::copyNonAttributePropertiesFromElement(const Element& source)
 {
     const HTMLTextFormControlElement& sourceElement = static_cast<const HTMLTextFormControlElement&>(source);
     m_lastChangeWasUserEdit = sourceElement.m_lastChangeWasUserEdit;
     HTMLFormControlElement::copyNonAttributePropertiesFromElement(source);
 }
 
 } // namespace blink