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

Unified Diff: cc/trees/layer_tree_host.cc

Issue 1884863003: Non passive touch end or touch cancel listeners should not block scroll. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove API from ChromeClient Created 4 years, 8 months 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 | « cc/proto/layer_tree_host.proto ('k') | cc/trees/layer_tree_host_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host.cc
diff --git a/cc/trees/layer_tree_host.cc b/cc/trees/layer_tree_host.cc
index 365008b8cbe727feeb35c6e6c9f0014554a08328..eaffd6af828d0df67b942ad31e9e0c25a3dd6133 100644
--- a/cc/trees/layer_tree_host.cc
+++ b/cc/trees/layer_tree_host.cc
@@ -441,11 +441,14 @@ void LayerTreeHost::FinishCommitOnImplThread(LayerTreeHostImpl* host_impl) {
sync_tree->set_has_transparent_background(has_transparent_background_);
sync_tree->set_have_scroll_event_handlers(have_scroll_event_handlers_);
sync_tree->set_event_listener_properties(
- EventListenerClass::kTouch,
- event_listener_properties(EventListenerClass::kTouch));
+ EventListenerClass::kTouchStartOrMove,
+ event_listener_properties(EventListenerClass::kTouchStartOrMove));
sync_tree->set_event_listener_properties(
EventListenerClass::kMouseWheel,
event_listener_properties(EventListenerClass::kMouseWheel));
+ sync_tree->set_event_listener_properties(
+ EventListenerClass::kTouchEndOrCancel,
+ event_listener_properties(EventListenerClass::kTouchEndOrCancel));
if (page_scale_layer_.get() && inner_viewport_scroll_layer_.get()) {
sync_tree->SetViewportLayersFromIds(
@@ -1529,8 +1532,12 @@ void LayerTreeHost::ToProtobufForCommit(proto::LayerTreeHost* proto) {
proto->set_have_scroll_event_handlers(have_scroll_event_handlers_);
proto->set_wheel_event_listener_properties(static_cast<uint32_t>(
event_listener_properties(EventListenerClass::kMouseWheel)));
- proto->set_touch_event_listener_properties(static_cast<uint32_t>(
- event_listener_properties(EventListenerClass::kTouch)));
+ proto->set_touch_start_or_move_event_listener_properties(
+ static_cast<uint32_t>(
+ event_listener_properties(EventListenerClass::kTouchStartOrMove)));
+ proto->set_touch_end_or_cancel_event_listener_properties(
+ static_cast<uint32_t>(
+ event_listener_properties(EventListenerClass::kTouchEndOrCancel)));
proto->set_in_paint_layer_contents(in_paint_layer_contents_);
proto->set_id(id_);
proto->set_next_commit_forces_redraw(next_commit_forces_redraw_);
@@ -1599,9 +1606,14 @@ void LayerTreeHost::FromProtobufForCommit(const proto::LayerTreeHost& proto) {
EventListenerClass::kMouseWheel)] =
static_cast<EventListenerProperties>(
proto.wheel_event_listener_properties());
- event_listener_properties_[static_cast<size_t>(EventListenerClass::kTouch)] =
+ event_listener_properties_[static_cast<size_t>(
+ EventListenerClass::kTouchStartOrMove)] =
+ static_cast<EventListenerProperties>(
+ proto.touch_start_or_move_event_listener_properties());
+ event_listener_properties_[static_cast<size_t>(
+ EventListenerClass::kTouchEndOrCancel)] =
static_cast<EventListenerProperties>(
- proto.touch_event_listener_properties());
+ proto.touch_end_or_cancel_event_listener_properties());
in_paint_layer_contents_ = proto.in_paint_layer_contents();
id_ = proto.id();
next_commit_forces_redraw_ = proto.next_commit_forces_redraw();
« no previous file with comments | « cc/proto/layer_tree_host.proto ('k') | cc/trees/layer_tree_host_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698