1071029 - 'contextmenu' events in Marionette are missing pageX and pageY values

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Chris Lord [:cwiiis]]

If you long-press on an element on a b2g device, the resulting 'contextmenu' event has 'pageX' and 'pageY' properties which are missing on the same event generated via Marionette.

Although I can't find documentation for the pageX/pageY properties, reading the code, it looks like the clientX/Y and screenX/Y properties are similarly missing.

Ideally, the Marionette event would have all three pairs so that it matched the behaviour of normal user-generated events in Gecko.

Comment 1

[Chris Lord [:cwiiis]]

Attachment: Fix contextmenu events in marionette

This fixes the issue locally.

Comment 2

[Malini Das [:mdas] - Away, not checking bugmail]

Comment on attachment 8493153 [details] [diff] [review]
Fix contextmenu events in marionette

Review of attachment 8493153 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks! This looks great.

Comment 3

[Chris Lord [:cwiiis]]

Pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/5d90266605a1

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/5d90266605a1

Comment 5

[Derek Harris [:DerekH] - Dont need info. Last day 6/5/15]

This is a testonly bug and no user-facing changes are made. Nothing to be verified here.