Add a fullscreen button on the top right corner of the inline video element. When user clicks the fullscreen button, browser goes to fullscreen mode to playing the video.
Created attachment 80261 [details] first try
Created attachment 80262 [details] screenshot
Comment on attachment 80261 [details] first try Palatial! r=me
Comment on attachment 80261 [details] first try Clearing flags on attachment: 80261 Committed r76873: <http://trac.webkit.org/changeset/76873>
All reviewed patches have been landed. Closing bug.
This bug requires the changes from Bug 52822 (Clean up the Media Controls CSS for Qt) and Bug 35427 (trivial fix for missing default control) to be applied. I also have the impression that these changes would be welcome on the 2.1.x branch as we'll have to support video/audio media on 2.1.x for some time and these changes would be welcome. Yi: what do you think? Have you tested the changes from Bug 52822 and Bug 35427 on the 2.1.x branch? They look like low risk changes to me.
(In reply to comment #6) > This bug requires the changes from Bug 52822 (Clean up the Media Controls CSS for Qt) and Bug 35427 (trivial fix for missing default control) to be applied. > > I also have the impression that these changes would be welcome on the 2.1.x branch as we'll have to support video/audio media on 2.1.x for some time and these changes would be welcome. > > Yi: what do you think? Have you tested the changes from Bug 52822 and Bug 35427 on the 2.1.x branch? They look like low risk changes to me. Hi, Ademar, I only have tested bug 52822 on html5-video-dev branch (http://gitorious.org/webkit/html5-video-dev) and it worked fine. I am agree with you that both changes are low risk and can be cherry-picked :)
Revision r76873 cherry-picked into qtwebkit-2.1.x with commit 138dcb6 <http://gitorious.org/webkit/qtwebkit/commit/138dcb6>