WebLocalFrameImpl::selectWordAroundPosition should select a word before space.

Source/core/editing/FrameSelectionTest.cpp

Index: Source/core/editing/FrameSelectionTest.cpp
diff --git a/Source/core/editing/FrameSelectionTest.cpp b/Source/core/editing/FrameSelectionTest.cpp
index 2aea63be379eab58fcfb09c86e29415cfadeeaae..0751de893c5e3ed980b47c50965ace8c6066a176 100644
--- a/Source/core/editing/FrameSelectionTest.cpp
+++ b/Source/core/editing/FrameSelectionTest.cpp
@@ -138,4 +138,25 @@ TEST_F(FrameSelectionTest, PaintCaretShouldNotLayout)
     EXPECT_EQ(startCount, layoutCount());
 }
 
+#define EXPECT_EQ_SELECTEDTEXT(text) \

[Yuta Kitamura, 2014/11/14 05:14:56]

SELECTED_TEXT

[yoichio, 2014/11/14 06:19:58]

Done.

+    EXPECT_EQ(text, WebString(selection().selectedText()).utf8());

[Yuta Kitamura, 2014/11/14 05:14:56]

1. Semicolon is unnecessary.

2. Usually, macros like this are defined as:

    #define FOO(bar) do { baz(bar); } while (false)

... but in this case that may be a little bit overkill, so I'm fine
with your version (modulo semicolon).

[yoichio, 2014/11/14 06:19:59]

Anyway, we need something bridges WTFString to utf8 for test output :)

+
+TEST_F(FrameSelectionTest, SelectWordAroundPosition)
+{
+    // "Foo Bar  Baz,"
+    RefPtrWillBeRawPtr<Text> text = document().createTextNode("Foo Bar&nbsp;&nbsp;Baz,");
+    document().body()->appendChild(text);
+    // "Fo|o Bar  Baz,"
+    EXPECT_TRUE(selection().selectWordAroundPosition(VisiblePosition(Position(text, 2))));
+    EXPECT_EQ_SELECTEDTEXT("Foo");
+    // "Foo| Bar  Baz,"
+    EXPECT_TRUE(selection().selectWordAroundPosition(VisiblePosition(Position(text, 3))));
+    EXPECT_EQ_SELECTEDTEXT("Foo");
+    // "Foo Bar | Baz,"
+    EXPECT_FALSE(selection().selectWordAroundPosition(VisiblePosition(Position(text, 13))));
+    // "Foo Bar  Baz|,"
+    EXPECT_TRUE(selection().selectWordAroundPosition(VisiblePosition(Position(text, 22))));
+    EXPECT_EQ_SELECTEDTEXT("Baz");
+}
+
 }