Bug 43529

Summary: [Qt] can't replay a video via Phonon
Product: WebKit Reporter: SH <sukhyun.kim>
Component: MediaAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WONTFIX    
Severity: Normal CC: kling, menard
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Test Case
none
Media file needed to play
none
Proposed Patch : replaying a video
kling: review-
Proposed Patch : replaying a video vestbo: review-

SH
Reported 2010-08-04 18:34:28 PDT
I installed ogg codec and tested html5 video tag using QtWebKit on Window XP. A html below works once although loop attribute is in. <video width="720" height="480" controls loop="loop"> <source src="animation_effect.ogv" /> </video> And if a play button is pressed after finishing play, a video does not replay.
Attachments
Test Case (114 bytes, text/html)
2010-08-29 20:26 PDT, SH
no flags
Media file needed to play (317.50 KB, video/ogg)
2010-08-29 20:27 PDT, SH
no flags
Proposed Patch : replaying a video (1.14 KB, patch)
2010-08-31 01:02 PDT, SH
kling: review-
Proposed Patch : replaying a video (1.18 KB, patch)
2010-09-06 02:47 PDT, SH
vestbo: review-
SH
Comment 1 2010-08-29 20:26:10 PDT
Created attachment 65874 [details] Test Case If a play button is pressed after finishing play, a video does not replay
SH
Comment 2 2010-08-29 20:27:29 PDT
Created attachment 65875 [details] Media file needed to play
SH
Comment 3 2010-08-31 01:02:09 PDT
Created attachment 66019 [details] Proposed Patch : replaying a video
Andreas Kling
Comment 4 2010-09-03 05:49:52 PDT
Comment on attachment 66019 [details] Proposed Patch : replaying a video > + [Qt] attributes of Video Tag This makes no sense, please change it (and the bug title) to something relevant. > + No new tests. (OOPS!) Oops indeed, it should be easy to make a manual test for this. If you cannot (or this is already covered by an existing test), please change the above line to say "No new test because $REASON" r- because of the lack of test. Change itself looks reasonable.
SH
Comment 5 2010-09-06 02:47:44 PDT
Created attachment 66616 [details] Proposed Patch : replaying a video
Tor Arne Vestbø
Comment 6 2010-09-06 03:42:07 PDT
Comment on attachment 66616 [details] Proposed Patch : replaying a video This should be handled by the following snippet from HTMLMediaElement::playInternal() if (endedPlayback()) { ExceptionCode unused; seek(0, unused); } Why isn't it? Also, the test should be in the form of a layout-test if there are none testing for this case (there should be). FYI the phonon backend is not really maintained, you'd be better off looking at the QtMultimedia backend and helping out there.
Alexis Menard (darktears)
Comment 7 2011-04-11 12:31:36 PDT
I'm closing the bug because the author didn't came back and our focus is QtMultimedia support.
Note You need to log in before you can comment on or make changes to this bug.