Relax text validation in FrameSelection::selectWordAroundPosition

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"
+#include "core/dom/NodeList.h"
 #include "core/dom/Text.h"
 #include "core/editing/EditingTestBase.h"
 #include "core/editing/FrameCaret.h"
 #include "core/editing/SelectionController.h"
 #include "core/frame/FrameView.h"
 #include "core/html/HTMLBodyElement.h"
 #include "core/input/EventHandler.h"
 #include "core/layout/LayoutBlock.h"
 #include "core/paint/PaintInfo.h"
 #include "core/paint/PaintLayer.h"
@@ skipping 99 matching lines @@
   EXPECT_EQ_SELECTED_TEXT("Foo");
   // "Foo Bar | Baz,"
   EXPECT_FALSE(selection().selectWordAroundPosition(
       createVisiblePosition(Position(text, 13))));
   // "Foo Bar  Baz|,"
   EXPECT_TRUE(selection().selectWordAroundPosition(
       createVisiblePosition(Position(text, 22))));
   EXPECT_EQ_SELECTED_TEXT("Baz");
 }
 
+// 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()->childNodes()->item(2);

[yosin_UTC9, 2017/03/01 03:31:03]

document().body()->firstChild()->lastChild() is easier and we don't need to
include "NodeList.h"

[yoichio, 2017/03/01 03:42:09]

Done.

+  EXPECT_TRUE(selection().selectWordAroundPosition(
+      createVisiblePosition(Position(baz, 2))));
+  // TODO(yoichio): We should select only "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()
                                .collapse(PositionInFlatTree(host, 0))
                                .extend(PositionInFlatTree(document().body(), 2))
@@ skipping 96 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