Range: node offsets should be unsigned.

third_party/WebKit/Source/core/editing/InputMethodControllerTest.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/editing/InputMethodController.h"
 
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
 #include "core/dom/Range.h"
 #include "core/editing/Editor.h"
@@ skipping 157 matching lines @@
 
 TEST_F(InputMethodControllerTest, SetCompositionFromExistingText) {
   Element* div = insertHTMLElement(
       "<div id='sample' contenteditable>hello world</div>", "sample");
 
   Vector<CompositionUnderline> underlines;
   underlines.push_back(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0));
   controller().setCompositionFromExistingText(underlines, 0, 5);
 
   Range* range = controller().compositionRange();
-  EXPECT_EQ(0, range->startOffset());
-  EXPECT_EQ(5, range->endOffset());
+  EXPECT_EQ(0u, range->startOffset());
+  EXPECT_EQ(5u, range->endOffset());
 
   PlainTextRange plainTextRange(PlainTextRange::create(*div, *range));
   EXPECT_EQ(0u, plainTextRange.start());
   EXPECT_EQ(5u, plainTextRange.end());
 }
 
 TEST_F(InputMethodControllerTest, SetCompositionAfterEmoji) {
   // "trophy" = U+1F3C6 = 0xF0 0x9F 0x8F 0x86 (UTF8).
   Element* div = insertHTMLElement(
       "<div id='sample' contenteditable>&#x1f3c6</div>", "sample");
@@ skipping 187 matching lines @@
   // Creates a div with one leading new line char. The new line char is hidden
   // from the user and IME, but is visible to InputMethodController.
   Element* div = insertHTMLElement(
       "<div id='sample' contenteditable>\nhello world</div>", "sample");
 
   Vector<CompositionUnderline> underlines;
   underlines.push_back(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0));
   controller().setCompositionFromExistingText(underlines, 0, 5);
 
   Range* range = controller().compositionRange();
-  EXPECT_EQ(1, range->startOffset());
-  EXPECT_EQ(6, range->endOffset());
+  EXPECT_EQ(1u, range->startOffset());
+  EXPECT_EQ(6u, range->endOffset());
 
   PlainTextRange plainTextRange(PlainTextRange::create(*div, *range));
   EXPECT_EQ(0u, plainTextRange.start());
   EXPECT_EQ(5u, plainTextRange.end());
 }
 
 TEST_F(InputMethodControllerTest,
        SetCompositionFromExistingTextWithInvalidOffsets) {
   insertHTMLElement("<div id='sample' contenteditable>test</div>", "sample");
 
@@ skipping 791 matching lines @@
 
   controller().commitText(String("string"), underlines, 0);
 
   ASSERT_EQ(1u, document().markers().markers().size());
 
   EXPECT_EQ(9u, document().markers().markers()[0]->startOffset());
   EXPECT_EQ(15u, document().markers().markers()[0]->endOffset());
 }
 
 }  // namespace blink