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

Unified Diff: content/browser/renderer_host/input/input_router_impl.cc

Issue 2734563002: Re-enable passthrough touch event queue. (Closed)
Patch Set: Created 3 years, 10 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
Index: content/browser/renderer_host/input/input_router_impl.cc
diff --git a/content/browser/renderer_host/input/input_router_impl.cc b/content/browser/renderer_host/input/input_router_impl.cc
index 686d0ef6dedfe90ed2797bc7376b74bd421a0351..308f0707b24814803bba46f19141d3e2d9a252ab 100644
--- a/content/browser/renderer_host/input/input_router_impl.cc
+++ b/content/browser/renderer_host/input/input_router_impl.cc
@@ -93,11 +93,10 @@ InputRouterImpl::InputRouterImpl(IPC::Sender* sender,
base::FeatureList::IsEnabled(
features::kTouchpadAndWheelScrollLatching)),
gesture_event_queue_(this, this, config.gesture_config),
- device_scale_factor_(1.f) {
- // TODO(dtapuska): Figure out regression caused by activating
- // the passthrough queue. crbug.com/697871
- if (false &&
- base::FeatureList::IsEnabled(features::kRafAlignedTouchInputEvents)) {
+ device_scale_factor_(1.f),
+ raf_aligned_touch_enabled_(
+ base::FeatureList::IsEnabled(features::kRafAlignedTouchInputEvents)) {
+ if (raf_aligned_touch_enabled_) {
touch_event_queue_.reset(
new PassthroughTouchEventQueue(this, config.touch_config));
} else {
@@ -380,7 +379,8 @@ void InputRouterImpl::OfferToHandlers(const WebInputEvent& input_event,
if (OfferToClient(input_event, latency_info))
return;
- bool should_block = WebInputEventTraits::ShouldBlockEventStream(input_event);
+ bool should_block = WebInputEventTraits::ShouldBlockEventStream(
+ input_event, raf_aligned_touch_enabled_);
OfferToRenderer(input_event, latency_info,
should_block
? InputEventDispatchType::DISPATCH_TYPE_BLOCKING

Powered by Google App Engine
This is Rietveld 408576698