1704875 - Don't synthesize events on menus in browser/components/extensions/test/browser/head.js

Status: RESOLVED FIXED

(WebExtensions :: General, task)

Comment 1

[Markus Stange [:mstange]]

Attachment: Bug 1704875 - Don't synthesize events on menus in browser/components/extensions/test/browser/head.js. r=robwu

Comment 2

[Markus Stange [:mstange]]

Attachment: Bug 1704875 - For now, keep synthesizing events on menus if a mouse button is set. r=robwu

This test checks that the mouse button is correctly propagated when clicking menu items.
Now that the closeExtensionContextMenu helper function uses activateItem, the mouse
button is discarded and this test fails. I have filed bug 1704879 for the activateItem
limitation.
As a workaround, temporarily change this test to manually synthesize events.
This will allow us to land the head.js patch without breaking the test for non-native menus.
Once bug 1704879 is fixed, we can revert this patch again.

Depends on D111673

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7ac3b6ec22fb
Don't synthesize events on menus in browser/components/extensions/test/browser/head.js. r=robwu
https://hg.mozilla.org/integration/autoland/rev/e48984c73090
For now, keep synthesizing events on menus if a mouse button is set. r=robwu

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/7ac3b6ec22fb
https://hg.mozilla.org/mozilla-central/rev/e48984c73090