Closed
Bug 1373234
Opened 7 years ago
Closed 6 years ago
Intermittent browser/components/extensions/test/browser/test-oop-extensions/browser_ext_menus.js | Test timed out | no tasks awaiting on messages - Got ["click"], expected [] | Extension left running at test shutdown | Found a tab
Categories
(WebExtensions :: Untriaged, defect, P5)
WebExtensions
Untriaged
Tracking
(firefox-esr60 fixed, firefox62 fixed, firefox63 fixed)
RESOLVED
FIXED
mozilla63
People
(Reporter: intermittent-bug-filer, Assigned: robwu)
References
Details
(Keywords: intermittent-failure)
Attachments
(2 files)
Filed by: rvandermeulen [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=107195573&repo=mozilla-beta https://queue.taskcluster.net/v1/task/eNpMpHnhQ6ew1fxMVVfP5A/runs/0/artifacts/public/logs/live_backing.log
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Updated•7 years ago
|
Priority: -- → P5
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Updated•6 years ago
|
Product: Toolkit → WebExtensions
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → rob
Status: NEW → ASSIGNED
Comment 56•6 years ago
|
||
mozreview-review |
Comment on attachment 8994799 [details] Bug 1373234 - Avoid races in tests involving menu creation https://reviewboard.mozilla.org/r/259346/#review266364 ::: commit-message-0b9c0:3 (Diff revision 1) > +browser.menus.create and browser.contextMenus.create are asynchronous > +APIs. For compatibility reasons, they cannot return a promise (since > +they already return an integer). browser.menus.create should have been a promise, I guess it's too late. ::: browser/components/extensions/test/browser/browser_ext_contextMenus.js:23 (Diff revision 1) > browser.contextMenus.create({ > id: "clickme-image", > title: "Click me!", > contexts: ["image"], > }); > + browser.contextMenus.onClicked.addListener(() => {}); what is the point of this line? ::: browser/components/extensions/test/browser/browser_ext_contextMenus.js:29 (Diff revision 1) > id: "clickme-page", > title: "Click me!", > contexts: ["page"], > - }); > + }, () => { > - browser.contextMenus.onClicked.addListener(() => {}); > - browser.test.notifyPass(); > + browser.test.notifyPass(); This is actually not a test pass. Change all of these to sendmessage. ::: browser/components/extensions/test/browser/browser_ext_contextMenus_icons.js:38 (Diff revision 1) > > browser.contextMenus.create({ > title: "child", > - }); > + }, () => { > - > - browser.test.notifyPass("contextmenus-icons"); > + browser.test.notifyPass("contextmenus-icons"); not a pass ::: browser/components/extensions/test/browser/browser_ext_contextMenus_icons.js:127 (Diff revision 1) > id: "contextmenu-child3", > icons: { > 18: "green_icon.png", > }, > + }, () => { > + browser.test.notifyPass("extra-contextmenu-items-added"); not a pass
Attachment #8994799 -
Flags: review?(mixedpuppy) → review+
Comment hidden (mozreview-request) |
Comment 58•6 years ago
|
||
mozreview-review |
Comment on attachment 8994905 [details] Bug 1373234 - Small cleanup in browser_ext_contextMenus*.js tests https://reviewboard.mozilla.org/r/259420/#review266444
Attachment #8994905 -
Flags: review?(mixedpuppy) → review+
Comment 59•6 years ago
|
||
Pushed by rob@robwu.nl: https://hg.mozilla.org/integration/autoland/rev/8cd5c902c0f8 Avoid races in tests involving menu creation r=mixedpuppy https://hg.mozilla.org/integration/autoland/rev/cae08ffe4804 Small cleanup in browser_ext_contextMenus*.js tests r=mixedpuppy
Comment 60•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8cd5c902c0f8 https://hg.mozilla.org/mozilla-central/rev/cae08ffe4804
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Comment hidden (Intermittent Failures Robot) |
Updated•6 years ago
|
status-firefox62:
--- → affected
status-firefox-esr60:
--- → affected
Comment 62•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr60/rev/66a7064bfc9f https://hg.mozilla.org/releases/mozilla-esr60/rev/71e5720baa20
Flags: in-testsuite+
Comment 63•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/0713cd1a0cb9 https://hg.mozilla.org/releases/mozilla-beta/rev/1da2d2a309c3
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 65•6 years ago
|
||
The failure in comment 64 belongs to bug 1493090 .
See Also: → 1493090
You need to log in
before you can comment on or make changes to this bug.
Description
•