604410 - Pause button should change to Play button at EOV

Status: VERIFIED DUPLICATE of bug 611994

(Core :: Audio/Video, defect)

Description

[u279076]

Build identifier: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b8pre) Gecko/20101013 Firefox/4.0b8pre

After playing an OGG video to its end, the play/pause button remains as a pause button, visually.  Clicking it starts playback from the beginning (as it should).  Visually, when a video finishes playing, the button should visually change to the Play state.

Steps to Reproduce:
1. Play http://videos.mozilla.org/firefox/3.6/getpersonas.ogv
2. Let the video reach the end
3. Notice the play/pause button state

Result:
Button appears as Pause

Expected Result:
Button appears as Play

Comment 1

[(mostly gone) XtC4UaLL [:xtc4uall]]

This is a Regression against 1.9.2 and Firefox 4 Beta 6 (tested on WinXP).

Comment 2

[Chris Pearce [:cpearce (Not reading bugmail)]]

This regressed in this range:

http://hg.mozilla.org/mozilla-central/pushloghtml?startdate=2010-09-13+04&enddate=2010-09-14+04

Candidate causes:

7aab25634114	Chris Double — Bug 571822 - Fire timeupdate event less frequently than once per frame - r=kinetik a=blocking2.0
33f5bd479735	Chris Double — Bug 464398 - Stop HTML media elements from firing events in bfcache - r=roc a=blocking2.0
eea753ee0156	Matthew Gregan — Bug 584615 - Fix for video controls direction test failure. r=kinetik a=blocking2.0
3da55bab7796	Chris Double — Bug 584615 - Make media progress events be 'simple' Events, not 'progress' Events - r=roc,cpearce,dolske,kinetik a=blocking2.0
a8cbb8e0287c	Matthew Gregan — Bug 596111 - Don't assume first WebM track is video. Seek using mVideoTrack (or mAudioTrack if there's no video). r=doublec
fd21140bec7b	Matthew Gregan — Bug 570904 - Implement buffered support for WebM. r=roc
ffd7d74232e6	Matthew Gregan — Bug 570904 - Update libnestegg. r=doublec

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

This regressed in 7aab25634114, "Fire timeupdate event less frequently than once per frame".

Comment 4

[Justin Dolske [:Dolske]]

Hmm, maybe bug 611994 fixed this?

Comment 5

[Chris Pearce [:cpearce (Not reading bugmail)]]

(In reply to comment #4)
> Hmm, maybe bug 611994 fixed this?

Yup, good spotting.