1704569 - Stop synthesizing events on context menus in various tests, round 1

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, task)

Description

[Markus Stange [:mstange]]

This bug will have the first set of patches to tweak various tests so that the can test what they want to test without synthesizing events on menus, which is impossible with native context menus (see bug 1704211).

Comment 1

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/base/content/test/contextMenu/browser_view_image.js. r=Gijs

Depends on D111669

Comment 2

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/base/content/test/about/browser_aboutNewTab_bookmarksToolbar.js. r=Gijs

Depends on D111670

Comment 3

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in various "reopen in container" tests, and add missing awaits. r=Gijs

Depends on D111671

Comment 4

[Markus Stange [:mstange]]

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

Depends on D111672

Comment 5

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/base/content/test/contextMenu/browser_contextmenu_save_blocked.js. r=Gijs

Depends on D111673

Comment 6

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/base/content/test/keyboard/browser_bookmarks_shortcut.js. r=Gijs

Depends on D111674

Comment 7

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/components/places/tests/browser/browser_library_bookmark_pages.js. r=Gijs

Depends on D111675

Comment 8

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/components/places/tests/browser/browser_toolbar_library_open_recent.js. r=Gijs

Depends on D111676

Comment 9

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/components/search/test/browser/browser_oneOffContextMenu_setDefault.js. r=Gijs

Depends on D111678

Comment 10

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/modules/test/browser/browser_UsageTelemetry_interaction.js. r=Gijs

Depends on D111679

Comment 11

[Markus Stange [:mstange]]

https://treeherder.mozilla.org/jobs?repo=try&revision=3b33f7340db0b1faee4369bf182c8037d531eaea

Comment 12

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in dom/html/test/browser_content_contextmenu_userinput.js. r=saschanaz

Comment 13

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in toolkit/components/pictureinpicture/tests/browser_contextMenu.js. r=mconley

Comment 14

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in toolkit/components/printing/tests/browser_print_context_menu.js. r=emalysz

Comment 15

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Use activateItem in browser/components/places/tests/browser/browser_default_bookmark_location.js. r=Gijs

Comment 16

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/base/content/test/pageActions-proton/browser_PageActions_removeExtension.js. r=mak

Comment 17

[Markus Stange [:mstange]]

Attachment: Bug 1704569 - Don't synthesize events on menus in browser/base/content/test/permissions/browser_temporary_permissions_navigation.js. r=Gijs

Comment 18

[Phabricator Automation]

Comment on attachment 9215179 [details]
Bug 1704569 - Don't synthesize events on menus in browser/components/places/tests/browser/browser_library_bookmark_pages.js. r=Gijs

Revision D111676 was moved to bug 1704837. Setting attachment 9215179 [details] to obsolete.

Comment 19

[Phabricator Automation]

Comment on attachment 9215176 [details]
Bug 1704569 - Don't synthesize events on menus in browser/components/extensions/test/browser/head.js. r=robwu

Revision D111673 was moved to bug 1704691. Setting attachment 9215176 [details] to obsolete.

Comment 20

[Phabricator Automation]

Comment on attachment 9215174 [details]
Bug 1704569 - Don't synthesize events on menus in browser/base/content/test/about/browser_aboutNewTab_bookmarksToolbar.js. r=Gijs

Revision D111671 was moved to bug 1704691. Setting attachment 9215174 [details] to obsolete.

Comment 21

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f828ac57927e
Don't synthesize events on menus in browser/base/content/test/contextMenu/browser_view_image.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/47061b7c2764
Don't synthesize events on menus in various "reopen in container" tests, and add missing awaits. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/da5566ea330e
Don't synthesize events on menus in browser/base/content/test/contextMenu/browser_contextmenu_save_blocked.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/c3a5ad2fe9d6
Don't synthesize events on menus in browser/base/content/test/keyboard/browser_bookmarks_shortcut.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/13d84b56fa91
Don't synthesize events on menus in browser/components/places/tests/browser/browser_toolbar_library_open_recent.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/630d671f5948
Don't synthesize events on menus in browser/components/search/test/browser/browser_oneOffContextMenu_setDefault.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/1d00b11169a6
Don't synthesize events on menus in browser/modules/test/browser/browser_UsageTelemetry_interaction.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/19169eec884e
Don't synthesize events on menus in dom/html/test/browser_content_contextmenu_userinput.js. r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/32c4f6d98c82
Don't synthesize events on menus in toolkit/components/pictureinpicture/tests/browser_contextMenu.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/5f7c5575a5b7
Don't synthesize events on menus in toolkit/components/printing/tests/browser_print_context_menu.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/56f346b8b80e
Use activateItem in browser/components/places/tests/browser/browser_default_bookmark_location.js. r=Gijs
https://hg.mozilla.org/integration/autoland/rev/8d03d72be0bc
Don't synthesize events on menus in browser/base/content/test/pageActions-proton/browser_PageActions_removeExtension.js. r=mak
https://hg.mozilla.org/integration/autoland/rev/0f1af3f5f896
Don't synthesize events on menus in browser/base/content/test/permissions/browser_temporary_permissions_navigation.js. r=Gijs

Comment 22

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/f828ac57927e
https://hg.mozilla.org/mozilla-central/rev/47061b7c2764
https://hg.mozilla.org/mozilla-central/rev/da5566ea330e
https://hg.mozilla.org/mozilla-central/rev/c3a5ad2fe9d6
https://hg.mozilla.org/mozilla-central/rev/13d84b56fa91
https://hg.mozilla.org/mozilla-central/rev/630d671f5948
https://hg.mozilla.org/mozilla-central/rev/1d00b11169a6
https://hg.mozilla.org/mozilla-central/rev/19169eec884e
https://hg.mozilla.org/mozilla-central/rev/32c4f6d98c82
https://hg.mozilla.org/mozilla-central/rev/5f7c5575a5b7
https://hg.mozilla.org/mozilla-central/rev/56f346b8b80e
https://hg.mozilla.org/mozilla-central/rev/8d03d72be0bc
https://hg.mozilla.org/mozilla-central/rev/0f1af3f5f896