[Android] Select text when stylus first button is pressed.

content/browser/renderer_host/render_widget_host_view_android.cc

 // Copyright (c) 2012 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 "content/browser/renderer_host/render_widget_host_view_android.h"
 
 #include <android/bitmap.h>
 
 #include "base/android/sys_utils.h"
 #include "base/basictypes.h"
@@ skipping 43 matching lines @@
 #include "content/public/browser/render_view_host.h"
 #include "content/public/common/content_switches.h"
 #include "gpu/command_buffer/client/gles2_interface.h"
 #include "gpu/config/gpu_driver_bug_workaround_type.h"
 #include "skia/ext/image_operations.h"
 #include "third_party/khronos/GLES2/gl2.h"
 #include "third_party/khronos/GLES2/gl2ext.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "ui/base/android/window_android.h"
 #include "ui/base/android/window_android_compositor.h"
+#include "ui/events/event_constants.h"
 #include "ui/events/gesture_detection/gesture_config_helper.h"
 #include "ui/events/gesture_detection/motion_event.h"
 #include "ui/gfx/android/device_display_info.h"
 #include "ui/gfx/android/java_bitmap.h"
 #include "ui/gfx/display.h"
 #include "ui/gfx/screen.h"
 #include "ui/gfx/size_conversions.h"
 
 namespace content {
 
@@ skipping 113 matching lines @@
           switches::kDisableOverscrollEdgeEffect)),
       overscroll_effect_(OverscrollGlow::Create(overscroll_effect_enabled_)),
       gesture_provider_(CreateGestureProviderConfig(), this),
       flush_input_requested_(false),
       accelerated_surface_route_id_(0),
       using_synchronous_compositor_(SynchronousCompositorImpl::FromID(
                                         widget_host->GetProcess()->GetID(),
                                         widget_host->GetRoutingID()) != NULL),
       frame_evictor_(new DelegatedFrameEvictor(this)),
       locks_on_frame_count_(0),
-      observing_root_window_(false) {
+      observing_root_window_(false),
+      gesture_text_selection_triggered_(false) {
   host_->SetView(this);
   SetContentViewCore(content_view_core);
   ImageTransportFactoryAndroid::AddObserver(this);
 }
 
 RenderWidgetHostViewAndroid::~RenderWidgetHostViewAndroid() {
   ImageTransportFactoryAndroid::RemoveObserver(this);
   SetContentViewCore(NULL);
   DCHECK(ack_callbacks_.empty());
   if (resource_collection_.get())
@@ skipping 338 matching lines @@
 void RenderWidgetHostViewAndroid::OnSmartClipDataExtracted(
     const base::string16& result) {
   // Custom serialization over IPC isn't allowed normally for security reasons.
   // Since this feature is only used in (single-process) WebView, there are no
   // security issues. Enforce that it's only called in single process mode.
   CHECK(RenderProcessHost::run_renderer_in_process());
   if (content_view_core_)
     content_view_core_->OnSmartClipDataExtracted(result);
 }
 
+bool RenderWidgetHostViewAndroid::ShouldTriggerOrUntriggerTextSelection(
+    const ui::MotionEvent& event) {
+  // Currently we aren't supporting stylus + touch case.
+  if (event.GetPointerCount() == 0 ||

[jdduke (slow), 2014/06/23 15:09:59]

Nit: GetPointerCount() should never be 0, so I would make this a DCHECK.

[Changwan Ryu, 2014/06/24 09:29:48]

Done.

+      event.GetToolType(0) != ui::MotionEvent::TOOL_TYPE_STYLUS)
+    return false;
+  const bool is_button_pressed =

[jdduke (slow), 2014/06/23 15:09:59]

Maybe rename to |is_secondary_button_pressed|?

[Changwan Ryu, 2014/06/24 09:29:48]

Done.

+      (event.GetButtonState() & ui::MotionEvent::BUTTON_SECONDARY) != 0;
+  const bool is_action_up = event.GetAction() == ui::MotionEvent::ACTION_UP;

[jdduke (slow), 2014/06/23 15:09:59]

Hmm, this is a little confusing, why do we check for ACTION_UP when this
function is only called in the if (ACTION_DOWN) case? I think we can remove this
check and just check the tool type and the secondary button state, and make this
function static (or rather, just move it to the anonymous namespace).

[Changwan Ryu, 2014/06/24 09:29:48]

Done.

+  return (is_button_pressed && !is_action_up) ||
+      (!is_button_pressed && gesture_text_selection_triggered_ && is_action_up);
+}
+
+bool RenderWidgetHostViewAndroid::IsGestureTextSelectionTriggered() const {
+  return gesture_text_selection_triggered_;
+}
+
 bool RenderWidgetHostViewAndroid::OnTouchEvent(
     const ui::MotionEvent& event) {
   if (!host_)
     return false;
 
   if (!gesture_provider_.OnTouchEvent(event))
     return false;
 
+  if (event.GetAction() == ui::MotionEvent::ACTION_DOWN) {
+    gesture_text_selection_triggered_ =
+        ShouldTriggerOrUntriggerTextSelection(event);
+  }
+
+  if (gesture_text_selection_triggered_) {
+    const bool event_consumed = false;
+    gesture_provider_.OnTouchEventAck(event_consumed);
+    if (event.GetAction() == ui::MotionEvent::ACTION_UP ||

[jdduke (slow), 2014/06/23 15:09:59]

As I said, we can't reset the state using ACTION_UP and ACTION_CANCEL, because
there are timeout events like GESTURE_TAP that might get triggered after an
ACTION_UP (because of the 300ms tap delay on desktop sites).  It should be
sufficient to reset the flag only on ACTION_DOWN.

[Changwan Ryu, 2014/06/24 09:29:48]

Removed.

+        event.GetAction() == ui::MotionEvent::ACTION_CANCEL)
+      gesture_text_selection_triggered_ = false;
+    return true;
+  }
+
   // Short-circuit touch forwarding if no touch handlers exist.
   if (!host_->ShouldForwardTouchEvent()) {
     const bool event_consumed = false;
     gesture_provider_.OnTouchEventAck(event_consumed);
     return true;
   }
 
   SendTouchEvent(CreateWebTouchEventFromMotionEvent(event));
   return true;
 }
@@ skipping 709 matching lines @@
 
 void RenderWidgetHostViewAndroid::RunAckCallbacks() {
   while (!ack_callbacks_.empty()) {
     ack_callbacks_.front().Run();
     ack_callbacks_.pop();
   }
 }
 
 void RenderWidgetHostViewAndroid::OnGestureEvent(
     const ui::GestureEventData& gesture) {
+  if (gesture_text_selection_triggered_) {
+    if (gesture.type() == ui::ET_GESTURE_LONG_PRESS) {
+      // Exits the text selection mode until ACTION_DOWN resets it.
+      gesture_text_selection_triggered_ = false;

[jdduke (slow), 2014/06/23 15:09:59]

Hmm, I don't think we want to reset this here. Can't we just send a SelectWord
message or something to that effect?

[Changwan Ryu, 2014/06/24 09:29:48]

Fixed to call SelectWord instead.
But reset is still needed because action_move events will most likely happen
after a long press, and those events will reselect or unselect text.

+    } else {
+      return;

[jdduke (slow), 2014/06/23 15:09:59]

I thought Samsung wanted to be able to highlight a word using either TAP or
LONG_PRESS? Is that not the case? 

[Changwan Ryu, 2014/06/24 09:29:48]

Samsung confirmed that it's only for LONG PRESS.

+    }
+  }
   SendGestureEvent(CreateWebGestureEventFromGestureEventData(gesture));
 }
 
 void RenderWidgetHostViewAndroid::OnCompositingDidCommit() {
   RunAckCallbacks();
 }
 
 void RenderWidgetHostViewAndroid::OnDetachCompositor() {
   DCHECK(content_view_core_);
   DCHECK(!using_synchronous_compositor_);
@@ skipping 144 matching lines @@
   results->availableRect = display.work_area();
   results->deviceScaleFactor = display.device_scale_factor();
   results->orientationAngle = display.RotationAsDegree();
   gfx::DeviceDisplayInfo info;
   results->depth = info.GetBitsPerPixel();
   results->depthPerComponent = info.GetBitsPerComponent();
   results->isMonochrome = (results->depthPerComponent == 0);
 }
 
 } // namespace content