Closed Bug 916399 Opened 12 years ago Closed 10 years ago

Intermittent test_timeupdate_small_files.html | Test timed out

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME
Milestone:
mozilla32
Tracking Flags:
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)

916399_fix.patch
1.81 KB, patch
cpearce
: review+
Details | Diff | Splinter Review
+++ 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: 11 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: 11 years ago → 10 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: