1264343 - test_peerConnection_scaleResolution.html times out on Android

Status: RESOLVED FIXED

(Core :: WebRTC: Audio/Video, defect, P2)

Description

[Dan Minor [:dminor]]

This test times out on Autophone and is also disabled on the emulators.

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

It would help to know how far they get (logs, perhaps with higher log levels)

Comment 2

[Dan Minor [:dminor]]

This test uses both VP8 and H.264. H.264 isn't supported in our test environment for Android, VP8 works fine.

Comment 3

[Dan Minor [:dminor]]

Attachment: Bug 1264343 - Only run test_peerConnection_scaleResolution.html with VP8 codec on Android;

We don't currently support H.264 on Android in automation, but we can improve
our test coverage by running the VP8 portion of this test in the meantime.

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

Comment 4

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

Comment on attachment 8857452 [details]
Bug 1264343 - Only run test_peerConnection_scaleResolution.html with VP8 codec on Android;

https://reviewboard.mozilla.org/r/129440/#review132082

::: dom/media/tests/mochitest/test_peerConnection_scaleResolution.html:80
(Diff revision 1)
>        })
>      })
>      .catch(generateErrorCallback());
>    }
>  
> +  if (navigator.appVersion.indexOf("Android") == -1) {

Might want to use `includes` rather than `indexOf`.

Comment 5

[Dan Minor [:dminor]]

Comment on attachment 8857452 [details]
Bug 1264343 - Only run test_peerConnection_scaleResolution.html with VP8 codec on Android;

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

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/538302e567b6
Only run test_peerConnection_scaleResolution.html with VP8 codec on Android; r=jib

Comment 7

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

https://hg.mozilla.org/mozilla-central/rev/538302e567b6