Fix bug causing TextIterator to duplicate leading spaces

third_party/WebKit/Source/core/editing/FrameSelectionTest.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/FrameSelection.h"
 
 #include <memory>
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
@@ skipping 134 matching lines @@
 
 // crbug.com/657996
 TEST_F(FrameSelectionTest, SelectWordAroundPosition2) {
   setBodyContent(
       "<p style='width:70px; font-size:14px'>foo bar<em>+</em> baz</p>");
   // "foo bar
   //  b|az"
   Node* const baz = document().body()->firstChild()->lastChild();
   EXPECT_TRUE(selection().selectWordAroundPosition(
       createVisiblePosition(Position(baz, 2))));
-  // TODO(yoichio): We should select only "baz".
-  EXPECT_EQ_SELECTED_TEXT(" baz");
+  EXPECT_EQ_SELECTED_TEXT("baz");
 }
 
 TEST_F(FrameSelectionTest, ModifyExtendWithFlatTree) {
   setBodyContent("<span id=host></span>one");
   setShadowContent("two<content></content>", "host");
   Element* host = document().getElementById("host");
   Node* const two = FlatTreeTraversal::firstChild(*host);
   // Select "two" for selection in DOM tree
   // Select "twoone" for selection in Flat tree
   selection().setSelection(SelectionInFlatTree::Builder()
@@ skipping 120 matching lines @@
                                .build());
   EXPECT_TRUE(selection().isHandleVisible());
   selection().selectAll();
   EXPECT_TRUE(selection().isHandleVisible())
       << "If handles were present before"
          "selectAll. Then they should be present"
          "after it.";
 }
 
 }  // namespace blink