WebLocalFrameImpl::selectWordAroundPosition should select a word before space.

Source/core/editing/FrameSelection.cpp

 /*
  * Copyright (C) 2004, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 54 matching lines @@
 #include "core/rendering/HitTestResult.h"
 #include "core/rendering/InlineTextBox.h"
 #include "core/rendering/RenderLayer.h"
 #include "core/rendering/RenderPart.h"
 #include "core/rendering/RenderText.h"
 #include "core/rendering/RenderTheme.h"
 #include "core/rendering/RenderView.h"
 #include "platform/SecureTextInput.h"
 #include "platform/geometry/FloatQuad.h"
 #include "platform/graphics/GraphicsContext.h"
+#include "platform/text/UnicodeUtilities.h"
 #include "wtf/text/CString.h"
 #include <stdio.h>
 
 #define EDIT_DEBUG 0
 
 namespace blink {
 
 using namespace HTMLNames;
 
 static inline LayoutUnit NoXPosForVerticalArrowNavigation()
@@ skipping 1838 matching lines @@
 }
 
 void FrameSelection::scheduleVisualUpdate() const
 {
     if (!m_frame)
         return;
     if (Page* page = m_frame->page())
         page->animator().scheduleVisualUpdate();
 }
 
+bool FrameSelection::selectWordAroundPosition(const VisiblePosition& position)
+{
+    EWordSide wordSideList[2] = { RightWordIfOnBoundary, LeftWordIfOnBoundary };
+    for (auto& wordSide : wordSideList) {

[yosin_UTC9, 2014/11/14 04:19:51]

Q: Do we need |&|?

[yoichio, 2014/11/14 04:50:21]

Used EWordSide instead.
http://chromium-cpp.appspot.com/

+        VisiblePosition start = startOfWord(position, wordSide);
+        VisiblePosition end = endOfWord(position, wordSide);
+

[yosin_UTC9, 2014/11/14 04:19:51]

nit: We don't need to have a blank line.

[yoichio, 2014/11/14 04:50:22]

Done.

+        String text = plainText(start.deepEquivalent(), end.deepEquivalent());
+

[yosin_UTC9, 2014/11/14 04:19:51]

nit: We don't need to have a blank line.

[yoichio, 2014/11/14 04:50:21]

Done.

+        if (!text.isEmpty() && !isSeparator(text.characterStartingAt(0))) {
+            setSelection(VisibleSelection(start, end), WordGranularity);
+            return true;
+        }
+    }
+
+    return false;
+}
+
 }
 
 #ifndef NDEBUG
 
 void showTree(const blink::FrameSelection& sel)
 {
     sel.showTreeForThis();
 }
 
 void showTree(const blink::FrameSelection* sel)
 {
     if (sel)
         sel->showTreeForThis();
 }
 
 #endif