Last Comment Bug 679262 - Seeking to end of file during loadedmetadata event breaks further playback
: Seeking to end of file during loadedmetadata event breaks further playback
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Audio/Video (show other bugs)
: Trunk
: x86_64 Linux
: -- normal (vote)
: mozilla9
Assigned To: cajbir (:cajbir)
:
: Maire Reavy [:mreavy]
Mentors:
Depends on:
Blocks: 648595
  Show dependency treegraph
 
Reported: 2011-08-15 20:55 PDT by cajbir (:cajbir)
Modified: 2011-08-30 04:32 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
testcase (856 bytes, text/html)
2011-08-15 20:55 PDT, cajbir (:cajbir)
no flags Details
Fix (2.07 KB, patch)
2011-08-25 22:23 PDT, cajbir (:cajbir)
cpearce: review+
Details | Diff | Splinter Review

Description cajbir (:cajbir) 2011-08-15 20:55:44 PDT
Created attachment 553360 [details]
testcase

In the loadedmetadata event I seek to the duration. An ended event is never raised, and the controls still show the 'pause' icon as if playback is still going. Pressing play/pause does not restart the playback.

Steps to reproduce:
 
1) Load testcase with 'content/media/tests/big.wav' in the local directory.

What should happen:

1) The 'Play' icon in the controls should appear.

What does happen:

2) The 'Pause' icon remains displayed.

Then do:

1) Press Play/pause.

What should happen:

1) Playback should start from the beginning

What does happen:

1) currentTime stays at the duration and no playback occurs.
Comment 1 cajbir (:cajbir) 2011-08-25 22:23:41 PDT
Created attachment 555959 [details] [diff] [review]
Fix

Fix is to raise ended event if we seeked to the end of the file. Test is provided by re-enabling the 'todo' media fragment test that tests this issue.
Comment 3 Ed Morley [:emorley] 2011-08-30 04:32:17 PDT
http://hg.mozilla.org/mozilla-central/rev/9722cd5cd38f

Note You need to log in before you can comment on or make changes to this bug.