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

Side by Side Diff: content/browser/renderer_host/input/input_router_impl.cc

Issue 2722113007: Disable the passthrough touch event queue for now. (Closed)
Patch Set: Fix failing test Created 3 years, 9 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 unified diff | Download patch
« no previous file with comments | « no previous file | content/browser/renderer_host/input/input_router_impl_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/input/input_router_impl.h" 5 #include "content/browser/renderer_host/input/input_router_impl.h"
6 6
7 #include <math.h> 7 #include <math.h>
8 8
9 #include <utility> 9 #include <utility>
10 10
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 move_caret_pending_(false), 87 move_caret_pending_(false),
88 current_ack_source_(ACK_SOURCE_NONE), 88 current_ack_source_(ACK_SOURCE_NONE),
89 flush_requested_(false), 89 flush_requested_(false),
90 active_renderer_fling_count_(0), 90 active_renderer_fling_count_(0),
91 touch_scroll_started_sent_(false), 91 touch_scroll_started_sent_(false),
92 wheel_event_queue_(this, 92 wheel_event_queue_(this,
93 base::FeatureList::IsEnabled( 93 base::FeatureList::IsEnabled(
94 features::kTouchpadAndWheelScrollLatching)), 94 features::kTouchpadAndWheelScrollLatching)),
95 gesture_event_queue_(this, this, config.gesture_config), 95 gesture_event_queue_(this, this, config.gesture_config),
96 device_scale_factor_(1.f) { 96 device_scale_factor_(1.f) {
97 if (base::FeatureList::IsEnabled(features::kRafAlignedTouchInputEvents)) 97 // TODO(dtapuska): Figure out regression caused by activating
98 // the passthrough queue. crbug.com/697871
99 if (false &&
100 base::FeatureList::IsEnabled(features::kRafAlignedTouchInputEvents)) {
98 touch_event_queue_.reset( 101 touch_event_queue_.reset(
99 new PassthroughTouchEventQueue(this, config.touch_config)); 102 new PassthroughTouchEventQueue(this, config.touch_config));
100 else 103 } else {
101 touch_event_queue_.reset( 104 touch_event_queue_.reset(
102 new LegacyTouchEventQueue(this, config.touch_config)); 105 new LegacyTouchEventQueue(this, config.touch_config));
106 }
103 107
104 DCHECK(sender); 108 DCHECK(sender);
105 DCHECK(client); 109 DCHECK(client);
106 DCHECK(ack_handler); 110 DCHECK(ack_handler);
107 UpdateTouchAckTimeoutEnabled(); 111 UpdateTouchAckTimeoutEnabled();
108 } 112 }
109 113
110 InputRouterImpl::~InputRouterImpl() { 114 InputRouterImpl::~InputRouterImpl() {
111 } 115 }
112 116
(...skipping 552 matching lines...) Expand 10 before | Expand all | Expand 10 after
665 DCHECK(global_touch_position_.find(touch_point->id) == 669 DCHECK(global_touch_position_.find(touch_point->id) ==
666 global_touch_position_.end()); 670 global_touch_position_.end());
667 global_touch_position_[touch_point->id] = gfx::Point( 671 global_touch_position_[touch_point->id] = gfx::Point(
668 touch_point->screenPosition.x, touch_point->screenPosition.y); 672 touch_point->screenPosition.x, touch_point->screenPosition.y);
669 } 673 }
670 } 674 }
671 } 675 }
672 } 676 }
673 677
674 } // namespace content 678 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | content/browser/renderer_host/input/input_router_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698