1345095 - Intermittent dom/media/tests/mochitest/test_peerConnection_capturedVideo.html | Test attempted to use a flaky timeout value 180000

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect)

Description

[Treeherder Bug Filer]

Filed by: cbook [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=82135510&repo=autoland

https://queue.taskcluster.net/v1/task/MfzWrYmDQxmXzgUQJ0o0Fg/runs/0/artifacts/public/logs/live_backing.log

Comment 1

[Intermittent Failures Robot]

17 failures in 141 pushes (0.121 failures/push) were associated with this bug yesterday.  
Repository breakdown:
* oak: 4
* mozilla-central: 4
* mozilla-inbound: 3
* graphics: 3
* autoland: 3

Platform breakdown:
* linux64: 11
* linux32: 6

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1345095&startday=2017-03-07&endday=2017-03-07&tree=all

Comment 2

[JW Wang [:jwwang] [:jw_wang]]

Attachment: Bug 1345095 - ensure SimpleTest.js is loaded before all other scripts.

Scripts are not necessarily loaded in the same order of |document.head.appendChild(el)|.
We use 2-pass loading to ensure SimpleTest.js is loaded before other scripts start loading.

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

Comment 3

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Comment on attachment 8844764 [details]
Bug 1345095 - ensure SimpleTest.js is loaded before all other scripts.

https://reviewboard.mozilla.org/r/118084/#review120106

r=me minus noisy log statements.

::: dom/media/tests/mochitest/pc.js:1896
(Diff revision 1)
> +    dump(`loading script: ${script}\n`);
> -  el.src = script;
> +    el.src = script;
> -  document.head.appendChild(el);
> +    document.head.appendChild(el);
> -  return new Promise(r => { el.onload = r; el.onerror = r; });
> +    return new Promise(r => {
> +      var f = () => {
> +        dump(`script loaded: ${script}\n`);

Did you mean to leave these `dump` statements in here?

Comment 4

[JW Wang [:jwwang] [:jw_wang]]

Thanks! I will remove the dump() calls.

Comment 5

[JW Wang [:jwwang] [:jw_wang]]

Comment on attachment 8844764 [details]
Bug 1345095 - ensure SimpleTest.js is loaded before all other scripts.

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

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c44ae7588ae4
ensure SimpleTest.js is loaded before all other scripts. r=jib

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c44ae7588ae4

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/68168a79f3ff

Comment 9

[Intermittent Failures Robot]

30 failures in 790 pushes (0.038 failures/push) were associated with this bug in the last 7 days.   

** This failure happened more than 30 times this week! Resolving this bug is a high priority. **

** Try to resolve this bug as soon as possible. If unresolved for 2 weeks, the affected test(s) may be disabled. ** 

Repository breakdown:
* mozilla-inbound: 8
* graphics: 6
* autoland: 6
* mozilla-central: 5
* oak: 4
* try: 1

Platform breakdown:
* linux64: 20
* linux32: 10

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1345095&startday=2017-03-06&endday=2017-03-12&tree=all