Bug 1321182 - Register click handler before starting click test in browser_ext_contextMenus.js r?mixedpuppy draft
authorRob Wu <rob@robwu.nl>
Fri, 20 Jul 2018 17:43:42 +0200
changeset 820874 c9000b0c3af76274fd35ae052bf137e0c5c040f2
parent 820804 4f12d77b4f9b6adaf06615c1c8cdc14de836dc1a
push id116972
push userbmo:rob@robwu.nl
push dateFri, 20 Jul 2018 15:45:13 +0000
reviewersmixedpuppy
bugs1321182
milestone63.0a1
Bug 1321182 - Register click handler before starting click test in browser_ext_contextMenus.js r?mixedpuppy MozReview-Commit-ID: FkOjNm7Tgd5
browser/components/extensions/test/browser/browser_ext_contextMenus.js
--- a/browser/components/extensions/test/browser/browser_ext_contextMenus.js
+++ b/browser/components/extensions/test/browser/browser_ext_contextMenus.js
@@ -529,27 +529,27 @@ add_task(async function test_bookmark_co
         url,
         title,
         parentId: "toolbar_____",
       });
       await browser.contextMenus.create({
         title: "Get bookmark",
         contexts: ["bookmark"],
       });
-      browser.test.sendMessage("bookmark-created");
       browser.contextMenus.onClicked.addListener(async (info) => {
         browser.test.assertEq(newBookmark.id, info.bookmarkId, "Bookmark ID matches");
 
         let [bookmark] = await browser.bookmarks.get(info.bookmarkId);
         browser.test.assertEq(title, bookmark.title, "Bookmark title matches");
         browser.test.assertEq(url, bookmark.url, "Bookmark url matches");
         browser.test.assertFalse(info.hasOwnProperty("pageUrl"), "Context menu does not expose pageUrl");
         await browser.bookmarks.remove(info.bookmarkId);
         browser.test.sendMessage("test-finish");
       });
+      browser.test.sendMessage("bookmark-created");
     },
   });
   await extension.startup();
   await extension.awaitMessage("bookmark-created");
   let menu = await openChromeContextMenu(
     "placesContext",
     "#PersonalToolbar .bookmark-item:last-child");