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 47629e3f994e2a57798b77ab0ff62148b0b21829..de867e03ddecbab934d89a8ce1b75b5c94582fea 100644
--- a/third_party/WebKit/Source/core/editing/FrameSelection.cpp
+++ b/third_party/WebKit/Source/core/editing/FrameSelection.cpp
@@ -1033,14 +1033,6 @@ 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};
@@ -1052,7 +1044,7 @@ bool FrameSelection::selectWordAroundPosition(const VisiblePosition& position) {
     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())