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

Unified Diff: content/renderer/render_view_impl.cc

Issue 1877073003: Respect the order of input messages from browser to renderer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased Created 4 years, 7 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/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 95be092f5438a72a802f8c3605e94d86d9807349..944d6369132d0cfaf9e502db8b9fa0eb0ebf5624 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -659,7 +659,8 @@ RenderViewImpl::RenderViewImpl(CompositorDependencies* compositor_deps,
expected_content_intent_id_(0),
#endif
enumeration_completion_id_(0),
- session_storage_namespace_id_(params.session_storage_namespace_id) {
+ session_storage_namespace_id_(params.session_storage_namespace_id),
+ has_added_input_handler_(false) {
GetWidget()->set_owner_delegate(this);
}
@@ -2046,17 +2047,19 @@ void RenderViewImpl::initializeLayerTreeView() {
// (SnowLeopard, which has Aqua scrollbars which need synchronous updates).
use_threaded_event_handling = compositor_deps_->IsElasticOverscrollEnabled();
#endif
- if (use_threaded_event_handling) {
- RenderThreadImpl* render_thread = RenderThreadImpl::current();
- // render_thread may be NULL in tests.
- InputHandlerManager* input_handler_manager =
- render_thread ? render_thread->input_handler_manager() : NULL;
- if (input_handler_manager) {
- input_handler_manager->AddInputHandler(
- GetRoutingID(), rwc->GetInputHandler(), AsWeakPtr(),
- webkit_preferences_.enable_scroll_animator,
- UseGestureBasedWheelScrolling());
- }
+ if (!use_threaded_event_handling)
+ return;
+
+ RenderThreadImpl* render_thread = RenderThreadImpl::current();
+ // render_thread may be NULL in tests.
+ InputHandlerManager* input_handler_manager =
+ render_thread ? render_thread->input_handler_manager() : NULL;
+ if (input_handler_manager) {
+ input_handler_manager->AddInputHandler(
+ GetRoutingID(), rwc->GetInputHandler(), AsWeakPtr(),
+ webkit_preferences_.enable_scroll_animator,
+ UseGestureBasedWheelScrolling());
+ has_added_input_handler_ = true;
}
}
@@ -2209,6 +2212,10 @@ float RenderViewImpl::GetDeviceScaleFactorForTest() const {
return device_scale_factor_;
}
+bool RenderViewImpl::HasAddedInputHandler() const {
+ return has_added_input_handler_;
+}
+
gfx::Point RenderViewImpl::ConvertWindowPointToViewport(
const gfx::Point& point) {
blink::WebFloatRect point_in_viewport(point.x(), point.y(), 0, 0);
« content/renderer/render_frame_impl.cc ('K') | « content/renderer/render_view_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698