Range: node offsets should be unsigned.

third_party/WebKit/Source/core/dom/Range.h

 /*
  * (C) 1999 Lars Knoll (knoll@kde.org)
  * (C) 2000 Gunnstein Lye (gunnstein@netcom.no)
  * (C) 2000 Frederik Holljen (frederik.holljen@hig.no)
  * (C) 2001 Peter Kelly (pmk@post.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. 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
@@ skipping 38 matching lines @@
 class Text;
 
 class CORE_EXPORT Range final : public GarbageCollected<Range>,
                                 public ScriptWrappable {
   DEFINE_WRAPPERTYPEINFO();
 
  public:
   static Range* create(Document&);
   static Range* create(Document&,
                        Node* startContainer,
-                       int startOffset,
+                       unsigned startOffset,
                        Node* endContainer,
-                       int endOffset);
+                       unsigned endOffset);
   static Range* create(Document&, const Position&, const Position&);
   static Range* createAdjustedToTreeScope(const TreeScope&, const Position&);
 
   void dispose();
 
   Document& ownerDocument() const {
     DCHECK(m_ownerDocument);
     return *m_ownerDocument.get();
   }
   Node* startContainer() const { return m_start.container(); }
-  int startOffset() const { return m_start.offset(); }
+  unsigned startOffset() const { return m_start.offset(); }
   Node* endContainer() const { return m_end.container(); }
-  int endOffset() const { return m_end.offset(); }
+  unsigned endOffset() const { return m_end.offset(); }
 
   bool collapsed() const { return m_start == m_end; }
   bool isConnected() const;
 
   Node* commonAncestorContainer() const;
   static Node* commonAncestorContainer(const Node* containerA,
                                        const Node* containerB);
   void setStart(Node* container,
-                int offset,
+                unsigned offset,
                 ExceptionState& = ASSERT_NO_EXCEPTION);
   void setEnd(Node* container,
-              int offset,
+              unsigned offset,
               ExceptionState& = ASSERT_NO_EXCEPTION);
   void collapse(bool toStart);
   bool isNodeFullyContained(Node&) const;
-  bool isPointInRange(Node* refNode, int offset, ExceptionState&) const;
-  short comparePoint(Node* refNode, int offset, ExceptionState&) const;
+  bool isPointInRange(Node* refNode, unsigned offset, ExceptionState&) const;
+  short comparePoint(Node* refNode, unsigned offset, ExceptionState&) const;
   enum CompareResults {
     NODE_BEFORE,
     NODE_AFTER,
     NODE_BEFORE_AND_AFTER,
     NODE_INSIDE
   };
   enum CompareHow { kStartToStart, kStartToEnd, kEndToEnd, kEndToStart };
   short compareBoundaryPoints(unsigned how,
                               const Range* sourceRange,
                               ExceptionState&) const;
   static short compareBoundaryPoints(Node* containerA,
-                                     int offsetA,
+                                     unsigned offsetA,
                                      Node* containerB,
-                                     int offsetB,
+                                     unsigned offsetB,
                                      ExceptionState&);
   static short compareBoundaryPoints(const RangeBoundaryPoint& boundaryA,
                                      const RangeBoundaryPoint& boundaryB,
                                      ExceptionState&);
   bool boundaryPointsValid() const;
   bool intersectsNode(Node* refNode, ExceptionState&);
   void deleteContents(ExceptionState&);
   DocumentFragment* extractContents(ExceptionState&);
   DocumentFragment* cloneContents(ExceptionState&);
   void insertNode(Node*, ExceptionState&);
@@ skipping 43 matching lines @@
   void updateOwnerDocumentIfNeeded();
 
   // Expand range to a unit (word or sentence or block or document) boundary.
   // Please refer to https://bugs.webkit.org/show_bug.cgi?id=27632 comment #5
   // for details.
   void expand(const String&, ExceptionState&);
 
   ClientRectList* getClientRects() const;
   ClientRect* getBoundingClientRect() const;
 
-  static Node* checkNodeWOffset(Node*, int offset, ExceptionState&);
+  static Node* checkNodeWOffset(Node*, unsigned offset, ExceptionState&);
 
   DECLARE_TRACE();
 
  private:
   explicit Range(Document&);
   Range(Document&,
         Node* startContainer,
-        int startOffset,
+        unsigned startOffset,
         Node* endContainer,
-        int endOffset);
+        unsigned endOffset);
 
   void setDocument(Document&);
 
   void checkNodeBA(Node*, ExceptionState&) const;
   void checkExtractPrecondition(ExceptionState&);
   bool hasSameRoot(const Node&) const;
 
   enum ActionType { DELETE_CONTENTS, EXTRACT_CONTENTS, CLONE_CONTENTS };
   DocumentFragment* processContents(ActionType, ExceptionState&);
   static Node* processContentsBetweenOffsets(ActionType,
@@ skipping 30 matching lines @@
 using RangeVector = HeapVector<Member<Range>>;
 
 }  // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showTree(const blink::Range*);
 #endif
 
 #endif  // Range_h