Fix bug causing TextIterator to duplicate leading spaces

third_party/WebKit/Source/core/editing/FrameSelection.cpp

Index: third_party/WebKit/Source/core/editing/FrameSelection.cpp
diff --git a/third_party/WebKit/Source/core/editing/FrameSelection.cpp b/third_party/WebKit/Source/core/editing/FrameSelection.cpp
index d45285793773acea87241e7fdb7a205f92b7e381..6645c43dfa2f7bf940e0cd57ff4c516af4dea5c2 100644
--- a/third_party/WebKit/Source/core/editing/FrameSelection.cpp
+++ b/third_party/WebKit/Source/core/editing/FrameSelection.cpp
@@ -1034,26 +1034,17 @@ void FrameSelection::scheduleVisualUpdateForPaintInvalidationIfNeeded() const {
     frameView->scheduleVisualUpdateForPaintInvalidationIfNeeded();
 }
 
-static bool hasNonSeparatorCharacter(const String& text) {
-  for (unsigned i = 0; i < text.length(); i++) {
-    if (!isSeparator(text.characterStartingAt(i)))
-      return true;
-  }
-  return false;
-}
-
 bool FrameSelection::selectWordAroundPosition(const VisiblePosition& position) {
   static const EWordSide wordSideList[2] = {RightWordIfOnBoundary,
                                             LeftWordIfOnBoundary};
   for (EWordSide wordSide : wordSideList) {
     // TODO(yoichio): We should have Position version of |start/endOfWord|
     // for avoiding unnecessary canonicalization.
-    // Then we don't need |hasNonSeparatorCharacter|.
     VisiblePosition start = startOfWord(position, wordSide);
     VisiblePosition end = endOfWord(position, wordSide);
     String text =
         plainText(EphemeralRange(start.deepEquivalent(), end.deepEquivalent()));
-    if (!text.isEmpty() && hasNonSeparatorCharacter(text)) {
+    if (!text.isEmpty() && !isSeparator(text.characterStartingAt(0))) {
       setSelection(SelectionInDOMTree::Builder()
                        .collapse(start.toPositionWithAffinity())
                        .extend(end.deepEquivalent())