Bug 1454627 - Re-enable browser_ext_user_events.js (bug 1381305) draft
authorRob Wu <rob@robwu.nl>
Fri, 27 Jul 2018 18:52:21 +0200
changeset 823587 4b259c8ce29699000ef607494cbb1d32d194c252
parent 823465 87bcafe428a4ad6017e59b915581ae00aa863407
child 823588 d6615b72024bc50895196cf89ae932ef24daf171
push id117739
push userbmo:rob@robwu.nl
push dateFri, 27 Jul 2018 17:14:40 +0000
bugs1454627, 1381305
milestone63.0a1
Bug 1454627 - Re-enable browser_ext_user_events.js (bug 1381305) Re-enable the test that was disabled in bug 1381305 and fix the underlying issue that caused th intermittent failure in the first place. MozReview-Commit-ID: BL9wS2fogaf
browser/components/extensions/test/browser/browser-common.ini
browser/components/extensions/test/browser/browser_ext_user_events.js
--- a/browser/components/extensions/test/browser/browser-common.ini
+++ b/browser/components/extensions/test/browser/browser-common.ini
@@ -205,17 +205,16 @@ skip-if = os == 'mac' # Save as PDF not 
 [browser_ext_tabs_cookieStoreId.js]
 [browser_ext_tabs_update.js]
 [browser_ext_tabs_zoom.js]
 [browser_ext_tabs_update_url.js]
 [browser_ext_themes_icons.js]
 [browser_ext_themes_validation.js]
 [browser_ext_url_overrides_newtab.js]
 [browser_ext_user_events.js]
-skip-if = debug || os == "linux" #Bug 1381305
 [browser_ext_webRequest.js]
 [browser_ext_webNavigation_frameId0.js]
 [browser_ext_webNavigation_getFrames.js]
 [browser_ext_webNavigation_onCreatedNavigationTarget.js]
 [browser_ext_webNavigation_onCreatedNavigationTarget_contextmenu.js]
 [browser_ext_webNavigation_onCreatedNavigationTarget_named_window.js]
 [browser_ext_webNavigation_onCreatedNavigationTarget_subframe_window_open.js]
 [browser_ext_webNavigation_onCreatedNavigationTarget_window_open.js]
--- a/browser/components/extensions/test/browser/browser_ext_user_events.js
+++ b/browser/components/extensions/test/browser/browser_ext_user_events.js
@@ -42,31 +42,31 @@ add_task(async function testSources() {
         }
       }
 
       let tabs = await browser.tabs.query({active: true, currentWindow: true});
       await browser.pageAction.show(tabs[0].id);
 
       browser.pageAction.onClicked.addListener(() => request("bookmarks"));
       browser.browserAction.onClicked.addListener(() => request("tabs"));
-
-      browser.contextMenus.create({
-        id: "menu",
-        title: "test user events",
-        contexts: ["page"],
-      });
       browser.contextMenus.onClicked.addListener(() => request("webNavigation"));
 
       browser.test.onMessage.addListener(msg => {
         if (msg === "openOptionsPage") {
           browser.runtime.openOptionsPage();
         }
       });
 
-      browser.test.sendMessage("actions-ready");
+      browser.contextMenus.create({
+        id: "menu",
+        title: "test user events",
+        contexts: ["page"],
+      }, () => {
+        browser.test.sendMessage("actions-ready");
+      });
     },
 
     files: {
       "options.html": `<!DOCTYPE html>
         <html lang="en">
         <head>
           <meta charset="UTF-8">
           <script src="options.js"></script>