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

Issue 2500253002: scheduler: Handle mouse wheel event disposition correctly

Created:
4 years, 1 month ago by Sami
Modified:
4 years, 1 month ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2883
Project:
chromium
Visibility:
Public.

Description

scheduler: Handle mouse wheel event disposition correctly Previously the renderer scheduler would treat mouse wheel events as prioritized events, but failed to keep track of whether they were processed (and preventDefault()'ed) on the main thread or whether they turned into a smooth scroll gesture. Making this distinction is important because in the former case the main thread is doing custom input handling (which we should be careful about scheduling) and in the latter we are dealing with a regular scroll gesture handled either by the main or the compositor thread. This patch fixes the problem by adding explicit processing for the MouseWheel event and a number of tests which cover all the relevant scenarios. BUG=654765 Review-Url: https://codereview.chromium.org/2495593003 Cr-Commit-Position: refs/heads/master@{#431321} (cherry picked from commit 4867673a4d5bad7c3030ac7bab0ee5bf41a6f634)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+72 lines, -3 lines) Patch
M third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc View 1 chunk +10 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl_unittest.cc View 2 chunks +62 lines, -3 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698