Blink handle selection handle visibility

content/browser/renderer_host/input/stylus_text_selector.h

Index: content/browser/renderer_host/input/stylus_text_selector.h
diff --git a/content/browser/renderer_host/input/stylus_text_selector.h b/content/browser/renderer_host/input/stylus_text_selector.h
index 05d2f1803822ae3f4dbe27fea06ab87b211e5977..8b4bdfe53f2528e6e66ba4c04e9f704e94a8aabf 100644
--- a/content/browser/renderer_host/input/stylus_text_selector.h
+++ b/content/browser/renderer_host/input/stylus_text_selector.h
@@ -31,7 +31,6 @@ class CONTENT_EXPORT StylusTextSelectorClient {
   // (x0, y0) and (x1, y1) indicate the bounds of the initial selection.
   virtual void OnStylusSelectBegin(float x0, float y0, float x1, float y1) = 0;
   virtual void OnStylusSelectUpdate(float x, float y) = 0;
-  virtual void OnStylusSelectEnd() = 0;

[aelias_OOO_until_Jul13, 2016/10/07 02:40:20]

Begin/End pairs are a common pattern and it's weird to only have the Begin, even
if right now the End is just logging.  I'd prefer this be kept.

[amaralp, 2016/10/14 00:53:07]

Done.

   virtual void OnStylusSelectTap(base::TimeTicks time, float x, float y) = 0;
 };
 
@@ -67,7 +66,6 @@ class CONTENT_EXPORT StylusTextSelector : public ui::SimpleGestureListener {
   bool text_selection_triggered_;
   bool secondary_button_pressed_;
   bool dragging_;
-  bool dragged_;

[aelias_OOO_until_Jul13, 2016/10/07 02:40:20]

Please move this cleanup to separate patch to keep this one a closure of related
changes.

[amaralp, 2016/10/14 00:53:07]

Done.

   float anchor_x_;
   float anchor_y_;
   std::unique_ptr<ui::GestureDetector> gesture_detector_;