Bug 149112

Summary: [GTK] media controls does not show up when playing video finishes.
Product: WebKit Reporter: ChangSeok Oh <changseok>
Component: WebKitGTKAssignee: ChangSeok Oh <changseok>
Status: RESOLVED FIXED    
Severity: Normal CC: calvaris, cgarcia, clopez, commit-queue, gustavo, pnormand
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 149151    
Bug Blocks: 149396    
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

ChangSeok Oh
Reported 2015-09-13 22:10:49 PDT
1. go http://www.w3schools.com/html/html5_video.asp 2. play the video 3. move out a cursor from video. 4. wait until video finishes. GTK port does not show controls after playing video. This behavior is different from what Mac port does. They do show controls when playing video finishes. At least, we should update timeline when showing controls after playing video.
Attachments
Patch (5.04 KB, patch)
2015-09-14 21:35 PDT, ChangSeok Oh 		no flags
DetailsFormatted DiffDiff
Patch (5.11 KB, patch)
2015-09-14 22:11 PDT, ChangSeok Oh 		no flags
DetailsFormatted DiffDiff
Patch (5.27 KB, patch)
2015-09-16 00:11 PDT, ChangSeok Oh 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
ChangSeok Oh
Comment 1 2015-09-14 21:35:20 PDT
Created attachment 261169 [details] Patch
ChangSeok Oh
Comment 2 2015-09-14 22:11:00 PDT
Created attachment 261173 [details] Patch
Philippe Normand
Comment 3 2015-09-14 23:54:19 PDT
Xabier wdyt about this patch?
Xabier Rodríguez Calvar
Comment 4 2015-09-15 03:35:14 PDT
(In reply to comment #3) > Xabier wdyt about this patch? At a first glance it looks ok. I don't know why but I cannot render any page with video so I cannot give the patch a try. Tests need to run smoothly without any change and some smoke testing needs to be done with both audio only and video to ensure nothing is broken. That code is quite delicate :)
ChangSeok Oh
Comment 5 2015-09-15 04:11:34 PDT
(In reply to comment #4) > (In reply to comment #3) > > Xabier wdyt about this patch? > > At a first glance it looks ok. > > I don't know why but I cannot render any page with video so I cannot give > the patch a try. > > Tests need to run smoothly without any change and some smoke testing needs > to be done with both audio only and video to ensure nothing is broken. That > code is quite delicate :) FYI, I'm trying to fix the existing issues for media controls. This is a series patch. Please consider bug149111, bug149113 as well. =)
ChangSeok Oh
Comment 6 2015-09-16 00:11:10 PDT
Created attachment 261299 [details] Patch
ChangSeok Oh
Comment 7 2015-09-21 01:03:51 PDT
Comment on attachment 261299 [details] Patch Here we go!
WebKit Commit Bot
Comment 8 2015-09-21 01:53:20 PDT
Comment on attachment 261299 [details] Patch Clearing flags on attachment: 261299 Committed r190054: <http://trac.webkit.org/changeset/190054>
WebKit Commit Bot
Comment 9 2015-09-21 01:53:23 PDT
All reviewed patches have been landed. Closing bug.
Carlos Alberto Lopez Perez
Comment 10 2015-12-09 12:13:45 PST
You missed to update some baselines after this changes. I have done that on http://trac.webkit.org/changeset/193853
Note You need to log in before you can comment on or make changes to this bug.