Closed Bug 916399 Opened 11 years ago Closed 9 years ago

Intermittent test_timeupdate_small_files.html | Test timed out

Categories

(Core :: Audio/Video, defect)

ARM
Android
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME
mozilla32
Tracking Status
firefox29 --- wontfix
firefox30 --- fixed
firefox31 --- fixed
firefox32 --- fixed
firefox-esr24 --- unaffected
b2g-v1.2 --- unaffected
b2g-v1.3 --- unaffected
b2g-v1.3T --- unaffected
b2g-v1.4 --- fixed
b2g-v2.0 --- fixed

People

(Reporter: philor, Assigned: jwwang)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #786539 +++

+++ This bug was initially created as a clone of Bug #687972 +++

https://tbpl.mozilla.org/php/getParsedLog.php?id=27863105&tree=Mozilla-Central
Android 2.2 Armv6 Tegra mozilla-central opt test mochitest-3 on 2013-09-13 19:37:17 PDT for push 53d5e43e23cc
slave: tegra-363

19248 INFO TEST-PASS | /tests/content/media/test/test_timeupdate_small_files.html | [finished detodos.opus-6] Length of array should match number of running tests
19249 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_timeupdate_small_files.html | Test timed out.
19250 INFO TEST-END | /tests/content/media/test/test_timeupdate_small_files.html | finished in 312226ms
log fills up with 
10:36:58     INFO -  System JS : ERROR (null):0 - uncaught exception: 2147746065

Last messages: 
10:37:11     INFO -  5134 INFO TEST-INFO | /tests/content/media/test/test_timeupdate_small_files.html | http://mochi.test:8888/tests/content/media/test/gizmo.mp4 got progress
10:37:11     INFO -  5135 INFO TEST-INFO | /tests/content/media/test/test_timeupdate_small_files.html | http://mochi.test:8888/tests/content/media/test/gizmo.mp4 got loadedmetadata
10:37:11     INFO -  5136 INFO TEST-INFO | /tests/content/media/test/test_timeupdate_small_files.html | http://mochi.test:8888/tests/content/media/test/gizmo.mp4 got loadeddata
10:37:11     INFO -  5137 INFO TEST-INFO | /tests/content/media/test/test_timeupdate_small_files.html | http://mochi.test:8888/tests/content/media/test/gizmo.mp4 got canplay
10:37:11     INFO -  5138 INFO TEST-INFO | /tests/content/media/test/test_timeupdate_small_files.html | http://mochi.test:8888/tests/content/media/test/gizmo.mp4 got progress
10:37:11     INFO -  5139 INFO TEST-PASS | /tests/content/media/test/test_timeupdate_small_files.html | [finished detodos.opus-8] Length of array should match number of running tests
10:37:11     INFO -  5140 INFO TEST-INFO | /tests/content/media/test/test_timeupdate_small_files.html | http://mochi.test:8888/tests/content/media/test/gizmo.mp4 got progress
10:37:11     INFO -  5141 INFO TEST-INFO | /tests/content/media/test/test_timeupdate_small_files.html | http://mochi.test:8888/tests/content/media/test/gizmo.mp4 got stalled
10:37:11     INFO -  TEST-INFO | screenshot: exit status 0
10:37:11     INFO -  5142 INFO TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_timeupdate_small_files.html | Test timed out.
Chris: this bug went critical on 4/4 or 4/6 after many months of rare hits.  Lately it's #4 on the orange list and getting hit around 40% of pushes in Win7 or Win8 M1 debug.  Anything you can think of that landed around then that might have destabilized it?  Is the test safe from timing holes?
Flags: needinfo?(cpearce)
I'd guess it's Bug 986947, which landed on 4 April and added an mp3-in-mp4 file to the tests that run in test_timeupdate_small_files on Win7/8 but not other platforms due to the way they handle HTMLMediaElement.canPlayType().
Flags: needinfo?(cpearce)
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Depends on: 1001317
Attached patch 916399_fix.patch — — Splinter Review
1. enable test on Linux
2. autoplay fails to activate due to Bug 1001317, call play() instead.

try: https://tbpl.mozilla.org/?tree=Try&rev=b593dfe1a45e
no timeouts on test_timeupdate_small_files.html
Attachment #8413594 - Flags: review?(cpearce)
(In reply to JW Wang [:jwwang] from comment #333)
> Created attachment 8413594 [details] [diff] [review]
> 916399_fix.patch
> 
> 1. enable test on Linux
> 2. autoplay fails to activate due to Bug 1001317, call play() instead.
> 
> try: https://tbpl.mozilla.org/?tree=Try&rev=b593dfe1a45e
> no timeouts on test_timeupdate_small_files.html

Can we fix bug 1001317 instead?
(In reply to Chris Pearce (:cpearce) from comment #350)
> (In reply to JW Wang [:jwwang] from comment #333)
> > Created attachment 8413594 [details] [diff] [review]
> > 916399_fix.patch
> > 
> > 1. enable test on Linux
> > 2. autoplay fails to activate due to Bug 1001317, call play() instead.
> > 
> > try: https://tbpl.mozilla.org/?tree=Try&rev=b593dfe1a45e
> > no timeouts on test_timeupdate_small_files.html
> 
> Can we fix bug 1001317 instead?

Bug 1001317 takes some time to fix. I would rather have another test case to test the functionality of 'autoplay'.
(In reply to JW Wang [:jwwang] from comment #355)
> (In reply to Chris Pearce (:cpearce) from comment #350)
> > (In reply to JW Wang [:jwwang] from comment #333)
> > > Created attachment 8413594 [details] [diff] [review]
> > > 916399_fix.patch
> > > 
> > > 1. enable test on Linux
> > > 2. autoplay fails to activate due to Bug 1001317, call play() instead.
> > > 
> > > try: https://tbpl.mozilla.org/?tree=Try&rev=b593dfe1a45e
> > > no timeouts on test_timeupdate_small_files.html
> > 
> > Can we fix bug 1001317 instead?
> 
> Bug 1001317 takes some time to fix. I would rather have another test case to
> test the functionality of 'autoplay'.

Ok, that makes sense. We already have a test for autoplay; test_autoplay.html.
Attachment #8413594 - Flags: review?(cpearce) → review+
Keywords: checkin-needed
(In reply to Chris Pearce (:cpearce) from comment #359)
> Ok, that makes sense. We already have a test for autoplay;
> test_autoplay.html.

The test seems to only read/write the 'autoplay' attribute. I will add some event handlers to see if autoplay really kicks off and we can receive some play events.
https://hg.mozilla.org/mozilla-central/rev/57bb3f642291
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
(In reply to TBPL Robot from comment #368)

Comment 363 hasn't merged back from m-c to fx-team yet, so no worries on this one :)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to TBPL Robot from comment #401)

10:48:53     INFO -  19428 INFO Unable to initialize audio client: 800700b7.
10:48:53     INFO -  19429 INFO [Parent 3984] WARNING: AudioStream::OpenCubeb() b805c10 failed to init cubeb: file c:\builds\moz2_slave\m-in-w32-d-0000000000000000000\build\content\media\AudioStream.cpp, line 670
10:48:53     INFO -  19430 INFO [Parent 3984] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file c:\builds\moz2_slave\m-in-w32-d-0000000000000000000\build\content\media\AudioStream.cpp, line 550
Depends on: 1051658
Status: REOPENED → RESOLVED
Closed: 10 years ago9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.