Bug 1704569 - Don't synthesize events on menus in various "reopen in...

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

Differential Revision: https://phabricator.services.mozilla.com/D111672

browser/base/content/test/tabs/browser_multiselect_tabs_reopen_in_container.js

@@ -17,7 +17,7 @@ async function openTabMenuFor(tab) {
 }
 
 async function openReopenMenuForTab(tab) {
-  openTabMenuFor(tab);
+  await openTabMenuFor(tab);
 
   let reopenItem = tab.ownerDocument.getElementById(
     "context_reopenInContainer"
@@ -26,11 +26,7 @@ async function openReopenMenuForTab(tab) {
 
   let reopenMenu = reopenItem.getElementsByTagName("menupopup")[0];
   let reopenMenuShown = BrowserTestUtils.waitForEvent(reopenMenu, "popupshown");
-  EventUtils.synthesizeMouseAtCenter(
-    reopenItem,
-    { type: "mousemove" },
-    tab.ownerGlobal
-  );
+  reopenItem.openMenu(true);
   await reopenMenuShown;
 
   return reopenMenu;
@@ -56,7 +52,7 @@ function openTabInContainer(gBrowser, tab, reopenMenu, id) {
   let menuitem = reopenMenu.querySelector(
     `menuitem[data-usercontextid="${id}"]`
   );
-  EventUtils.synthesizeMouseAtCenter(menuitem, {}, menuitem.ownerGlobal);
+  reopenMenu.activateItem(menuitem);
   return tabPromise;
 }
 

browser/components/contextualidentity/test/browser/browser_reopenIn.js

@@ -21,7 +21,7 @@ async function openTabMenuFor(tab) {
 }
 
 async function openReopenMenuForTab(tab) {
-  openTabMenuFor(tab);
+  await openTabMenuFor(tab);
 
   let reopenItem = tab.ownerDocument.getElementById(
     "context_reopenInContainer"
@@ -30,11 +30,7 @@ async function openReopenMenuForTab(tab) {
 
   let reopenMenu = reopenItem.getElementsByTagName("menupopup")[0];
   let reopenMenuShown = BrowserTestUtils.waitForEvent(reopenMenu, "popupshown");
-  EventUtils.synthesizeMouseAtCenter(
-    reopenItem,
-    { type: "mousemove" },
-    tab.ownerGlobal
-  );
+  reopenItem.openMenu(true);
   await reopenMenuShown;
 
   return reopenMenu;
@@ -60,7 +56,7 @@ function openTabInContainer(gBrowser, reopenMenu, id) {
   let menuitem = reopenMenu.querySelector(
     `menuitem[data-usercontextid="${id}"]`
   );
-  EventUtils.synthesizeMouseAtCenter(menuitem, {}, menuitem.ownerGlobal);
+  reopenMenu.activateItem(menuitem);
   return tabPromise;
 }
 
@@ -128,12 +124,12 @@ add_task(async function testDisabled() {
     "Tab with No Container should be opened"
   );
 
-  openTabMenuFor(tab);
+  let tabMenu = await openTabMenuFor(tab);
   let reopenItem = document.getElementById("context_reopenInContainer");
   ok(reopenItem.hidden, "Reopen in Container item should be hidden");
 
   // Close the tab menu.
-  EventUtils.synthesizeKey("KEY_Escape");
+  tabMenu.hidePopup();
 
   BrowserTestUtils.removeTab(tab);
 });
@@ -155,14 +151,14 @@ add_task(async function testPrivateMode() {
     "Tab with No Container should be opened"
   );
 
-  openTabMenuFor(tab);
+  let tabMenu = await openTabMenuFor(tab);
   let reopenItem = privateWindow.document.getElementById(
     "context_reopenInContainer"
   );
   ok(reopenItem.hidden, "Reopen in Container item should be hidden");
 
   // Close the tab menu.
-  EventUtils.synthesizeKey("KEY_Escape");
+  tabMenu.hidePopup();
 
   await BrowserTestUtils.closeWindow(privateWindow);
 });

browser/components/contextualidentity/test/browser/head.js

@@ -13,22 +13,20 @@ async function openTabMenuFor(tab) {
 }
 
 async function openReopenMenuForTab(tab) {
-  openTabMenuFor(tab);
+  await openTabMenuFor(tab);
 
   let reopenItem = tab.ownerDocument.getElementById(
     "context_reopenInContainer"
   );
   ok(!reopenItem.hidden, "Reopen in Container item should be shown");
 
-  const menuPopup = tab.ownerDocument.getElementById(
-    "context_reopenInContainer"
-  ).menupopup;
+  const menuPopup = reopenItem.menupopup;
   const menuPopupPromise = BrowserTestUtils.waitForEvent(
     menuPopup,
     "popupshown"
   );
   info(`About to open a popup`);
-  menuPopup.openPopup();
+  reopenItem.openMenu(true);
   info(`Waiting for the menu popup promise`);
   await menuPopupPromise;
   info(`Awaited menu popup promise`);
@@ -40,7 +38,8 @@ function openTabInContainer(gBrowser, url, reopenMenu, id) {
   let menuitem = reopenMenu.querySelector(
     `menuitem[data-usercontextid="${id}"]`
   );
-  EventUtils.synthesizeMouseAtCenter(menuitem, {}, menuitem.ownerGlobal);
+  info(`about to activate item`);
+  reopenMenu.activateItem(menuitem);
   return tabPromise;
 }