[Contextual Search] Add support for tap on the selection.

ui/touch_selection/touch_selection_controller.cc

 // 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 "ui/touch_selection/touch_selection_controller.h"
 
 #include "base/auto_reset.h"
 #include "base/logging.h"
 #include "base/metrics/histogram_macros.h"
 
@@ skipping 133 matching lines @@
     if ((event_pos - GetStartPosition()).LengthSquared() <=
         (event_pos - GetEndPosition()).LengthSquared())
       return start_selection_handle_->WillHandleTouchEvent(event);
     else
       return end_selection_handle_->WillHandleTouchEvent(event);
   }
 
   return false;
 }
 
-void TouchSelectionController::OnLongPressEvent() {
+bool TouchSelectionController::WillHandleLongPressEvent(
+    const gfx::PointF& location) {
+  // If there is an active selection that was not triggered by a user gesture,
+  // allow showing the handles for that selection if a long-press occurs within
+  // the selection rect. Note that this hit test is at best a crude
+  // approximation, and may swallow taps that actually fall outside the
+  // real selection.
+  if (!is_selection_active_ && !is_insertion_active_) {
+    if (GetStartPosition() != GetEndPosition() &&
+        RectFBetweenSelectionBounds(start_, end_).Contains(location)) {
+      AllowShowingFromCurrentSelection();
+      return true;
+    }
+  }
+
   response_pending_input_event_ = LONG_PRESS;
   ShowSelectionHandlesAutomatically();
   ShowInsertionHandleAutomatically();
   ResetCachedValuesIfInactive();
+  return false;
 }
 
 void TouchSelectionController::AllowShowingFromCurrentSelection() {
   if (is_selection_active_ || is_insertion_active_)
     return;
 
   activate_selection_automatically_ = true;
   activate_insertion_automatically_ = true;
   if (GetStartPosition() != GetEndPosition())
     OnSelectionChanged();
   else if (start_orientation_ == TouchHandleOrientation::CENTER &&
            selection_editable_)
     OnInsertionChanged();
 }
 
-void TouchSelectionController::OnTapEvent() {
+bool TouchSelectionController::WillHandleTapEvent(const gfx::PointF& location) {
+  // If there is an active selection that was not triggered by a user gesture,
+  // allow showing the handles for that selection if the tap occurs within the
+  // selection rect. Note that this hit test is at best a crude approximation,
+  // and may swallow taps that actually fall outside the real selection.
+  if (!is_selection_active_ && !is_insertion_active_) {

[jdduke (slow), 2015/05/01 20:52:06]

Can we split this logic out into a shared function?

if (WillHandleTapOrLongPress(location))
  return true;

[Donn Denman, 2015/05/01 22:46:03]

Done.

+    if (GetStartPosition() != GetEndPosition() &&
+        RectFBetweenSelectionBounds(start_, end_).Contains(location)) {
+      AllowShowingFromCurrentSelection();
+      return true;
+    }
+  }
   response_pending_input_event_ = TAP;
   ShowInsertionHandleAutomatically();
   if (selection_empty_ && !show_on_tap_for_empty_editable_)
     DeactivateInsertion();
   ResetCachedValuesIfInactive();
+  return false;
 }
 
 void TouchSelectionController::HideAndDisallowShowingAutomatically() {
   response_pending_input_event_ = INPUT_EVENT_TYPE_NONE;
   DeactivateInsertion();
   DeactivateSelection();
   activate_insertion_automatically_ = false;
   activate_selection_automatically_ = false;
 }
 
@@ skipping 311 matching lines @@
     base::TimeDelta duration = base::TimeTicks::Now() - selection_start_time_;
     UMA_HISTOGRAM_CUSTOM_TIMES("Event.TouchSelection.WasDraggedDuration",
                                duration,
                                base::TimeDelta::FromMilliseconds(500),
                                base::TimeDelta::FromSeconds(60),
                                60);
   }
 }
 
 }  // namespace ui