488287 - "playing" event not triggered after the first play

Status: RESOLVED FIXED

(Core :: Audio/Video, defect, P2)

Description

[Paul Rouget [:paul]]

The video element should trigger a "play" event when it starts playing.
It works well with Firefox 3.1b3.

The nightlies doesn't work well. If you play the video a second time, the event is not triggered if the play operation is done from the JS API (video.play()) or from the controls. But it works from the context menu.

http://people.mozilla.com/~prouget/bugs/play_event.xhtml

* load the page
* click "video.play()"
* an alert shows that the "play" event is triggered
* wait the end of the video
* click "video.play()"
* the event is not triggered (but it is if you play the video from the context menu) and the video starts

Comment 1

[David Bolter [:davidb] (NeedInfo me for attention)]

Same on OS X.

Comment 2

[Matthew Gregan [:kinetik]]

"play" will only fire once, but "playing" should fire again when the media is replayed.  The media cache landing changed this behaviour, because we have only ever triggered "playing" as part of reloading the media, which no longer happens with the media cache.

There was some discussion about what the spec required, and the result of that is here:
http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2009-March/019012.html

Comment 3

[Matthew Gregan [:kinetik]]

Note that the mochitests were modified to account for the new media cache behaviour, so they should be changed back when this bug is fixed.

Comment 4

[alexander :surkov (:asurkov)]

This one might be related with bug 490114.

Comment 5

[Matthew Gregan [:kinetik]]

Attachment: patch v0

Change readyState to HAVE_CURRENT_DATA once playback ends.  Revert relevant tests back to how they were before bug 475441 landed.

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

http://hg.mozilla.org/mozilla-central/rev/4aa17bf8dcda

Comment 7

[Robert O'Callahan (:roc) (email my personal email if necessary)]

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/efb7fe220a31