New issue
Advanced search Search tips

Issue 800333 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

RWHIER::DispatchMouseEvent() uses wrong coords for SendMouseEnterOrLeaveEvents()

Project Member Reported by wjmaclean@chromium.org, Jan 9 2018

Issue description

DispatchMouseEvent() assumes event coords are in target-view coords (see 
https://cs.chromium.org/chromium/src/content/browser/renderer_host/render_widget_host_input_event_router.h?rcl=41223362de706e400ebf8adbeeb86bc7d3b6abab&l=179)

but SendMouseEnterOrLeaveEvents() assumes they are in root-view coords (see here:
https://cs.chromium.org/chromium/src/content/browser/renderer_host/render_widget_host_input_event_router.cc?rcl=41223362de706e400ebf8adbeeb86bc7d3b6abab&l=661).

Would it be simpler to require all RWHIER::DispatchXEvent() fcns take the events with root-view coords? (See proposed CL here: https://chromium-review.googlesource.com/c/chromium/src/+/855118)
 
Status: WontFix (was: Untriaged)
Since this seems to be working despite the apparent mis-match in event coords, marking this as wontfix.
Ha so we actually did have a bug for this... I believe it was fixed in https://chromium-review.googlesource.com/c/chromium/src/+/972671

Sign in to add a comment