On tap in editable node with some text complete text gets selected instead of showing insertion han…

On tap in editable node with some text complete text gets selected instead of showing insertion handler

On Longpress, m_start is set to Zero as wordside is set to
LeftWordIfOnBoundary. So the word left to this is getting selected.

To show paste popup instead of selecting,the word side to
RightWordIfOnBoundary is changed. So that m_start is set to end of
the word, and selection does not happen.

BUG=392101

[divya.bansal, 2014-07-08 09:19:18]

PTAL

[divya.bansal, 2014-07-25 16:51:21]

PTAL

[esprehn, 2014-08-19 02:10:40]

Tests? Have a screenshot of what you're changing here...

[yosin_UTC9, 2014-08-25 01:10:26]

Please add layout test for your change.
Thanks in advance.

https://codereview.chromium.org/376893002/diff/1/Source/core/editing/VisibleS...
File Source/core/editing/VisibleSelection.cpp (right):

https://codereview.chromium.org/376893002/diff/1/Source/core/editing/VisibleS...
Source/core/editing/VisibleSelection.cpp:364: if (isContentEditable() ||
(isEndOfLine(start) && !isStartOfLine(start) && isEndOfParagraph(start)))
This function should select word rather than collapsing selection. You may want
to change call site for long press to collapse selection to end of word.

[yosin_UTC9, 2014-09-25 02:16:03]

Let's implement touch related thing in EventHandler, rather than editing
component, which is independent from input and output.

For this case, it is in |selectClosestWordFromHitTestResult()|. 

We may want to have touch version of |selectClosestWordFromHitTestResult()| and
it should be smarter about @media (pointer:coarse) with considering zoom.

There is another patch for selecting word by tap: http://crrev.com/585963002