Show handles even when dragging from outside selectable region

content/browser/renderer_host/input/gesture_text_selector.cc

Index: content/browser/renderer_host/input/gesture_text_selector.cc
diff --git a/content/browser/renderer_host/input/gesture_text_selector.cc b/content/browser/renderer_host/input/gesture_text_selector.cc
index d9fa90d1f5e8d5c074bf7ae1f635812e7d1b82da..bed6acfa74682d14635e2b9d22aee80146a50a53 100644
--- a/content/browser/renderer_host/input/gesture_text_selector.cc
+++ b/content/browser/renderer_host/input/gesture_text_selector.cc
@@ -49,6 +49,7 @@ bool GestureTextSelector::OnGestureEvent(const ui::GestureEventData& gesture) {
       break;
     }
     case ui::ET_GESTURE_SCROLL_UPDATE: {
+      client_->ShowSelectionHandlesAutomatically();

[jdduke (slow), 2014/07/25 14:07:27]

Any idea why the call to |client_->ShowSelectionHandlesAutomatically();| from
ET_GESTURE_SCROLL_BEGIN is insufficient?

[jdduke (slow), 2014/07/25 14:25:03]

I wonder if the |Unselect()| call is triggering a new selection update which
"consumes" the automatic showing.  In any case, with this change I would remove
the |client_->ShowSelectionHandlesAutomatically()| call entirely from
ET_GESTURE_SCROLL_BEGIN.  With that, lgtm (although it would still be good to
verify the exact order of events making the previous solution insufficient), and
please validate the callback behavior in an existing test (or new one). 

I'm still wondering if it would be better to show the handles *after* the
dragging is finished, but that's a discussion for another day.

[Changwan Ryu, 2014/07/25 22:12:47]

Removed ShowSelectionHandlesAutomatically() from ET_GESTURE_SCROLL_BEGIN.

This only occurs when dragging starts in unselectable region, so I suspect that
we hide selection handles when it is not applicable (selected region is null). I
agree that showing handles on ET_GESTURE_SCROLL_END is much neater, but I'd like
to address that in the future. (Seems that refactoring is being done. :) )

On 2014/07/25 14:25:03, jdduke wrote:

       client_->SelectRange(anchor_x_, anchor_y_, gesture.x, gesture.y);
       break;
     }