1232334 - Autophone - Mdm failure - dom/media/test/test_bug879717.html

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Bob Clary [:bc] (inactive)]

https://treeherder.allizom.org/#/jobs?repo=try&exclusion_profile=false&group_state=expanded&author=[e-mail]

Samsung Galaxy S3 Android 4.0

>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on loadeddata of v1 (gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on play of v1 (gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on playing of v1 (gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on loadeddata of v2 (Captured gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on play of v2 (Captured gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on playing of v2 (Captured gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on loadedmetadata of v3 (Stream of gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on loadeddata of v3 (Stream of gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on playing of v3 (Stream of gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on ended of v3 (Stream of gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on ended of v2 (Captured gizmo.mp4), got NS_ERROR_NOT_AVAILABLE
>TEST-UNEXPECTED-FAIL | dom/media/test/test_bug879717.html | drawImage shouldn't throw an exception on ended of v1 (gizmo.mp4), got NS_ERROR_NOT_AVAILABLE

Only available in my local Autophone instance. If you would like to test patches, do a try build and ping me and I will run it locally. If you don't care about this, please just mark it WONTFIX.

Comment 1

[Eugen Sawin [:esawin]]

Any media test that creates multiple decoders is likely to fail on the Samsung S3 (bug 1214710) which explains the results in comment 0.

Running it on the Nexus devices seems to work fine (test cases pass) but then it crashes when playback has ended: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6c7492494e11&filter-tier=1&filter-tier=2&filter-tier=3&exclusion_profile=false

Limiting the test to a single media file fixes the crash: https://treeherder.mozilla.org/#/jobs?repo=try&revision=78d891256aa7&exclusion_profile=false&filter-tier=1&filter-tier=2&filter-tier=3

All media files pass the tests when tested in my setup (http://me73.com/media/draw.html).
There is something that triggers the crash when we chain the tests with MediaTestManager.runTests.

Comment 2

[Eugen Sawin [:esawin]]

The test now only fails on Nexus 4 Android 4.2.

Comment 3

[Eugen Sawin [:esawin]]

Attachment: 0001-Bug-1232334-Disallow-parallel-tests-on-Android.-r-sn.patch

Disabling parallel tests seems to fix it: https://treeherder.mozilla.org/#/jobs?repo=try&revision=c3d2f77fdb6a&exclusion_profile=false&filter-tier=1&filter-tier=2&filter-tier=3

I can't explain why, all the tests pass when executed separately.

Comment 4

[James Willcox (:snorp) ([e-mail]) (he/him)]

Comment on attachment 8725446 [details] [diff] [review]
0001-Bug-1232334-Disallow-parallel-tests-on-Android.-r-sn.patch

Review of attachment 8725446 [details] [diff] [review]:
-----------------------------------------------------------------

Do we have any idea why this helps? I'm a little worried we're covering up another bug here.

Comment 5

[Eugen Sawin [:esawin]]

Attachment: 0001-Bug-1232334-1.2-Only-set-context-on-successful-attac.patch

Don't set attached context if attaching to GL context fails.

This fixes the hang timeout on the Nexus 4:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9cc090d26a79&filter-tier=1&filter-tier=2&filter-tier=3&exclusion_profile=false

There was an intermittent timeout on the Nexus S Android 2.3, which never failed before for me, but it doesn't show the same characteristics, it looks like a regular timeout. We already request a longer timeout for GB, we can increase it should this become an reoccurring intermittent error.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1931aaadbec4

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/1931aaadbec4