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

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: fling layer 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
« no previous file with comments | « content/renderer/input/input_handler_manager.cc ('k') | ui/events/blink/input_handler_proxy_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..a4100162e2c0adec240cd4968f1069fa0aaabbb6 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_SCROLLING_LAYER;
result = DID_NOT_HANDLE;
break;
}
@@ -991,8 +998,9 @@ InputHandlerProxy::EventDisposition InputHandlerProxy::HandleTouchStart(
result = DID_HANDLE_NON_BLOCKING;
}
- bool is_fling_on_impl = fling_curve_ && !fling_may_be_active_on_main_thread_;
- if (result == DID_NOT_HANDLE && is_fling_on_impl)
+ bool is_flinging_on_impl =
+ fling_curve_ && !fling_may_be_active_on_main_thread_;
+ if (is_flinging_on_impl && is_touching_scrolling_layer)
result = DID_NOT_HANDLE_NON_BLOCKING_DUE_TO_FLING;
return result;
« no previous file with comments | « content/renderer/input/input_handler_manager.cc ('k') | ui/events/blink/input_handler_proxy_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698