Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(753)

Unified Diff: ui/events/blink/input_handler_proxy.cc

Issue 2471523002: Make touch events uncancelable during fling when they are on the current active scroll layer (Closed)
Patch Set: Rename variable Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/events/blink/input_handler_proxy.cc
diff --git a/ui/events/blink/input_handler_proxy.cc b/ui/events/blink/input_handler_proxy.cc
index ae52f930a7bb120dafa969c6e244855a4024e0e6..9768bf4818ae3e3a2d0ee428768488455447a60f 100644
--- a/ui/events/blink/input_handler_proxy.cc
+++ b/ui/events/blink/input_handler_proxy.cc
@@ -935,12 +935,19 @@ InputHandlerProxy::EventDisposition InputHandlerProxy::HandleGestureFlingStart(
InputHandlerProxy::EventDisposition InputHandlerProxy::HandleTouchStart(
const blink::WebTouchEvent& touch_event) {
EventDisposition result = DROP_EVENT;
+ bool is_touching_scrolling_layer = false;
for (size_t i = 0; i < touch_event.touchesLength; ++i) {
if (touch_event.touches[i].state != WebTouchPoint::StatePressed)
continue;
- if (input_handler_->DoTouchEventsBlockScrollAt(
+ cc::InputHandler::TouchStartEventListenerType event_listener_type =
+ input_handler_->EventListenerTypeForTouchStartAt(
gfx::Point(touch_event.touches[i].position.x,
- touch_event.touches[i].position.y))) {
+ touch_event.touches[i].position.y));
+ if (event_listener_type !=
+ cc::InputHandler::TouchStartEventListenerType::NO_HANDLER) {
+ is_touching_scrolling_layer =
+ event_listener_type == cc::InputHandler::TouchStartEventListenerType::
+ HANDLER_ON_FLINGING_LAYER;
result = DID_NOT_HANDLE;
break;
}
@@ -992,7 +999,7 @@ InputHandlerProxy::EventDisposition InputHandlerProxy::HandleTouchStart(
}
bool is_fling_on_impl = fling_curve_ && !fling_may_be_active_on_main_thread_;
- if (result == DID_NOT_HANDLE && is_fling_on_impl)
+ if (is_fling_on_impl && is_touching_scrolling_layer)
result = DID_NOT_HANDLE_NON_BLOCKING_DUE_TO_FLING;
return result;

Powered by Google App Engine
This is Rietveld 408576698