1232517 - Intermittent test_browserElement_NotifyChannel.html | App got audiochannel-interruption-begin. - Result logged after SimpleTest.finish()

Status: RESOLVED FIXED

(Firefox OS Graveyard :: AudioChannel, defect)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://treeherder.mozilla.org/logviewer.html#?job_id=6194204&repo=fx-team

 16:20:26     INFO -  18 INFO TEST-OK | dom/bindings/test/test_dom_xrays.html | took 11848ms
 16:20:26     INFO -  19 INFO TEST-START | dom/bindings/test/test_kill_longrunning_prerendered_content.xul
 16:20:37     INFO -  20 INFO TEST-OK | dom/bindings/test/test_kill_longrunning_prerendered_content.xul | took 9114ms
 16:20:37     INFO -  21 INFO TEST-START | dom/bindings/test/test_proxies_via_xray.html
 16:20:37     INFO -  22 INFO TEST-OK | dom/bindings/test/test_proxies_via_xray.html | took 4662ms
 16:20:37     INFO -  23 INFO TEST-START | dom/browser-element/mochitest/test_browserElement_NotifyChannel.html
 16:21:45     INFO -  24 INFO TEST-OK | dom/browser-element/mochitest/test_browserElement_NotifyChannel.html | took 68474ms
 16:21:57     INFO -  25 INFO TEST-UNEXPECTED-FAIL | dom/browser-element/mochitest/test_browserElement_NotifyChannel.html | App got audiochannel-interruption-begin. - Result logged after SimpleTest.finish()
 16:21:57     INFO -  26 INFO TEST-START | dom/cache/test/mochitest/test_chrome_constructor.html
 16:21:57     INFO -  27 INFO TEST-PASS | dom/cache/test/mochitest/test_chrome_constructor.html | storage should create cache
 16:21:57     INFO -  28 INFO TEST-PASS | dom/cache/test/mochitest/test_chrome_constructor.html | cache should contain response
 16:22:08     INFO -  29 INFO TEST-PASS | dom/cache/test/mochitest/test_chrome_constructor.html | storage should delete cache
 16:22:08     INFO -  30 INFO TEST-OK | dom/cache/test/mochitest/test_chrome_constructor.html | took 9948ms

Comment 1

[Alastor Wu [:alwu]]

Attachment: MozReview Request: Bug 1232517 - remove the event listener. r=baku.

Review commit: https://reviewboard.mozilla.org/r/29025/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/29025/

Comment 2

[Alastor Wu [:alwu]]

Comment on attachment 8701728 [details]
MozReview Request: Bug 1232517 - remove the event listener. r=baku.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/29025/diff/1-2/

Comment 3

[Alastor Wu [:alwu]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=2d7e59e2849f

Comment 4

[Alastor Wu [:alwu]]

Comment on attachment 8701728 [details]
MozReview Request: Bug 1232517 - remove the event listener. r=baku.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/29025/diff/2-3/

Comment 5

[Andrea Marchesini [:baku]]

Comment on attachment 8701728 [details]
MozReview Request: Bug 1232517 - remove the event listener. r=baku.

https://reviewboard.mozilla.org/r/29025/#review26339

::: dom/browser-element/mochitest/browserElement_NotifyChannel.js:52
(Diff revision 2)
> +    var request = navigator.mozApps.mgmt.uninstall(app);

app => aApp ?
or remove the param in uninstallApp.

Comment 6

[Alastor Wu [:alwu]]

Comment on attachment 8701728 [details]
MozReview Request: Bug 1232517 - remove the event listener. r=baku.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/29025/diff/2-3/

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/03c12499f78d

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/03c12499f78d