Closed
Bug 1360796
Opened 7 years ago
Closed 7 years ago
Intermittent browser/components/extensions/test/browser/browser_ext_tabs_executeScript_bad.js | Promise rejected, expecting rejection to match /Missing host permission/, got "Message manager disconnected": Should not execute in `data:` frame -
Categories
(WebExtensions :: Untriaged, defect)
WebExtensions
Untriaged
Tracking
(firefox55 fixed)
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: mixedpuppy)
Details
(Keywords: intermittent-failure, Whiteboard: [stockwell fixed])
Attachments
(1 file)
Filed by: philringnalda [at] gmail.com https://treeherder.mozilla.org/logviewer.html#?job_id=95392738&repo=autoland https://archive.mozilla.org/pub/firefox/tinderbox-builds/autoland-macosx64-debug/autoland_yosemite_r7-debug_test-mochitest-e10s-browser-chrome-1-bm107-tests1-macosx-build917.txt.gz
Comment hidden (Intermittent Failures Robot) |
Comment 2•7 years ago
|
||
this really picked up in the last week: https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1360796&startday=2017-04-24&endday=2017-05-05&tree=all a lot of debug, heavy on the osx side, all since April 29th! I did some retriggers here to see if the pattern is obvious, maybe this will help: https://treeherder.mozilla.org/#/jobs?repo=autoland&filter-searchStr=osx%20debug%20e10s-browser-chrome-1&tochange=05ea64eb8d85556e8f12af0335382cfa72fafce9&fromchange=796fdd50c2eb345382ec7ae3479dfdfece8cdf80&selectedJob=95388967 :andym, can you look at this or help find someone to look at this. This is becoming higher in priority as it is more frequent.
Flags: needinfo?(amckay)
Whiteboard: [stockwell needswork]
Comment 3•7 years ago
|
||
the retriggers are done, I suspect this backout of bug 1354682 is the root cause: https://hg.mozilla.org/integration/autoland/rev/fe58bf6ff79543d2ea940e631d55d8bc13c64d3b that is extensions related, maybe that will give a clue?
Comment hidden (Intermittent Failures Robot) |
Comment 5•7 years ago
|
||
Andy, this continues to fail at a very frequent failure rate, can you take a look at this real soon, or do you have a problem with us disabling this test until there is more time to look at it?
Comment hidden (Intermittent Failures Robot) |
Updated•7 years ago
|
Assignee: nobody → mixedpuppy
Flags: needinfo?(amckay)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 8•7 years ago
|
||
I think this is caused by an extra message being sent to the listener, it should be handled by being explicit about what messages the test listener handles. Try is running, we'll see.
Comment 9•7 years ago
|
||
mozreview-review |
Comment on attachment 8867834 [details] Bug 1360796 fix intermittent assert, https://reviewboard.mozilla.org/r/139356/#review142690 Checking for the correct messages seems like a good idea to me. ::: browser/components/extensions/test/browser/browser_ext_tabs_executeScript_bad.js:155 (Diff revision 1) > <script src="page.js"></script> > <iframe id="inherited" src="data:text/html;charset=utf-8,inherited"></iframe> > `, > "page.js": function() { > browser.test.onMessage.addListener((msg, url) => { > + if (msg === "navigate") { Nit: for consistency, could you make this the same as you do below where you return early if the wrong message is received, or vice versa?
Attachment #8867834 -
Flags: review?(mwein) → review+
Comment 10•7 years ago
|
||
mozreview-review |
Comment on attachment 8867834 [details] Bug 1360796 fix intermittent assert, https://reviewboard.mozilla.org/r/139356/#review142720 ::: browser/components/extensions/test/browser/browser_ext_tabs_executeScript_bad.js:155 (Diff revision 1) > <script src="page.js"></script> > <iframe id="inherited" src="data:text/html;charset=utf-8,inherited"></iframe> > `, > "page.js": function() { > browser.test.onMessage.addListener((msg, url) => { > + if (msg === "navigate") { The two extensions are each sent only one type of message, the message names were more for documenting the intention. ::: browser/components/extensions/test/browser/browser_ext_tabs_executeScript_bad.js:182 (Diff revision 1) > > browser.test.onMessage.addListener(async msg => { > + if (msg !== "execute") { > + return; > + } > browser.test.assertRejects( This is missing `await`, and could be the issue here.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 12•7 years ago
|
||
> > browser.test.onMessage.addListener(async msg => {
> > + if (msg !== "execute") {
> > + return;
> > + }
> > browser.test.assertRejects(
>
> This is missing `await`, and could be the issue here.
good catch, that is the core issue.
Comment 13•7 years ago
|
||
Pushed by mixedpuppy@gmail.com: https://hg.mozilla.org/integration/autoland/rev/7f3d55afb48e fix intermittent assert, r=mattw
Comment 14•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7f3d55afb48e
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Updated•7 years ago
|
Whiteboard: [stockwell needswork] → [stockwell fixed]
Comment hidden (Intermittent Failures Robot) |
Updated•6 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•