WebLocalFrameImpl::selectWordAroundPosition should select a word before space.

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 "config.h"
 #include "core/editing/FrameSelection.h"
 
 #include "bindings/core/v8/ExceptionStatePlaceholder.h"
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
@@ skipping 120 matching lines @@
         FrameView& frameView = dummyPageHolder().frameView();
         IntRect frameRect = frameView.frameRect();
         frameRect.setWidth(frameRect.width() + 1);
         frameRect.setHeight(frameRect.height() + 1);
         dummyPageHolder().frameView().setFrameRect(frameRect);
     }
     selection().paintCaret(nullptr, LayoutPoint(), LayoutRect());
     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");
 }
+
+}