Closed
Bug 1232334
Opened 9 years ago
Closed 9 years ago
Autophone - Mdm failure - dom/media/test/test_bug879717.html
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(firefox47 fixed, fennec+)
RESOLVED
FIXED
Firefox 47
People
(Reporter: bc, Assigned: esawin)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
https://treeherder.allizom.org/#/jobs?repo=try&exclusion_profile=false&group_state=expanded&author=bclary@mozilla.com 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.
Assignee: nobody → esawin
tracking-fennec: ? → +
Assignee | ||
Comment 1•9 years ago
|
||
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.
Assignee | ||
Comment 3•9 years ago
|
||
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.
Attachment #8725446 -
Flags: review?(snorp)
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.
Assignee | ||
Comment 5•9 years ago
|
||
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.
Attachment #8725446 -
Attachment is obsolete: true
Attachment #8725446 -
Flags: review?(snorp)
Attachment #8726265 -
Flags: review?(snorp)
Attachment #8726265 -
Flags: review?(snorp) → review+
Comment 7•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1931aaadbec4
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 47
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•