Closed Bug 495319 Opened 11 years ago Closed 11 years ago

Small vorbis file does not fire timeupdate event after play.

Categories

(Core :: Audio/Video, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
mozilla1.9.2a1

People

(Reporter: mikeal, Assigned: roc)

Details

(Keywords: verified1.9.1)

Attachments

(2 files)

We discovered this bug when trying to write an automated test for bug 461286.

The attached html file has some simple javascript in it that adds an event listener for timeupdate and updates the textContent of a div on the page. As you can see the event never fires.
Attached patch fixSplinter Review
This fixes it.
Attachment #380350 - Flags: review?(chris.double)
Attachment #380350 - Flags: review?(chris.double) → review+
Comment on attachment 380350 [details] [diff] [review]
fix

This may help us write tests. In itself it isn't super-important, but it is fairly trivial.
Comment on attachment 380350 [details] [diff] [review]
fix

a191=beltzner
Attachment #380350 - Flags: approval1.9.1? → approval1.9.1+
http://hg.mozilla.org/mozilla-central/rev/cf03f9d5bd95
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [needs 191 landing]
Flags: in-testsuite+
The testcase in comment 0 is still not firing timeupdate events.  Reopen/
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090602 Minefield/3.6a1pre

The timeupdate is firing, however the video breaks right after that and no more seeks/plays work.
Verified fix on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090603 Minefield/3.6a1pre
and Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1pre) Gecko/20090603 Shiretoko/3.5pre Ubiquity/0.1.5.  

The testcase is just a short video, but the timeupdates events is working here.
Status: RESOLVED → VERIFIED
Target Milestone: --- → mozilla1.9.2a1
You need to log in before you can comment on or make changes to this bug.