Closed Bug 1363253 Opened 3 years ago Closed 3 years ago
_devices _get _user _media _screen .js | recording-window-ended notification unexpected - Got -1, expected 0
59 bytes, text/x-review-board-request
Filed by: philringnalda [at] gmail.com https://treeherder.mozilla.org/logviewer.html#?job_id=97256621&repo=mozilla-central https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-central-win64-pgo/1494221402/mozilla-central_win8_64_test_pgo-mochitest-browser-chrome-7-bm119-tests1-windows-build33.txt.gz
Probably because we re-enabled tests in bug 1320994. Andreas, thoughts?
Priority: -- → P1
See Also: → 1324303
I did an analysis of where these errors could come from and found a hole in the test code. The patches currently up on mozreview are looking good so far on try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a8ab96666774e9f6215bdca025e23f372098d620
Assignee: nobody → pehrson
Status: NEW → ASSIGNED
See Also: → 1338038
Comment on attachment 8866772 [details] Bug 1363253 - Re-enable tests disabled due to gum events. https://reviewboard.mozilla.org/r/138382/#review142644 This patch doesn't do what it's commit message says.
Attachment #8866772 - Flags: review?(florian) → review-
Comment on attachment 8866773 [details] Bug 1363253 - Ignore events specific to preview window also when decrementing. https://reviewboard.mozilla.org/r/138384/#review142650 Great catch, thanks!
Attachment #8866773 - Flags: review?(florian) → review+
Comment on attachment 8866772 [details] Bug 1363253 - Re-enable tests disabled due to gum events. https://reviewboard.mozilla.org/r/138382/#review142644 Ah, I started on this before bug 1320994 landed and after a rebase this patch was basically moot.
Attachment #8866772 - Attachment is obsolete: true
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/e7c7370fdd54 Ignore events specific to preview window also when decrementing. r=florian
IIUC, this depends on bug 1320994 (which I assume is a bit risky for uplift)? So Fx54 is wontfix?
I cannot say for sure if this is dependent on bug 1320994. They're separate issues but without bug 1320994 this could perhaps cause other intermittent fallout. It could be worth trying. And this is test-only so it doesn't affect the product, just the intermittents.
Duplicate of this bug: 1324303
Duplicate of this bug: 1338038
We'll see what happens anyway! https://hg.mozilla.org/releases/mozilla-beta/rev/4f95bb8a858d
You need to log in before you can comment on or make changes to this bug.