Commit aac0a3fb authored by yi.4.shen@nokia.com's avatar yi.4.shen@nokia.com

2011-02-18 Yi Shen <yi.4.shen@nokia.com>

        Reviewed by Tor Arne Vestbø.

        Always display the media controls when requiresFullscreenForVideoPlayback() is true
        https://bugs.webkit.org/show_bug.cgi?id=54308

        For video element, it should have controls when
        Chrome::requiresFullscreenForVideoPlayback() is true.

        * html/HTMLMediaElement.cpp:
        (WebCore::HTMLMediaElement::controls):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@79085 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent bf5c462b
2011-02-18 Yi Shen <yi.4.shen@nokia.com>
Reviewed by Tor Arne Vestbø.
Always display the media controls when requiresFullscreenForVideoPlayback() is true
https://bugs.webkit.org/show_bug.cgi?id=54308
For video element, it should have controls when
Chrome::requiresFullscreenForVideoPlayback() is true.
* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::controls):
2011-02-18 Adrienne Walker <enne@google.com>
Reviewed by Kenneth Russell.
......
......@@ -1470,6 +1470,10 @@ bool HTMLMediaElement::controls() const
if (frame && !frame->script()->canExecuteScripts(NotAboutToExecuteScript))
return true;
// always show controls for video when fullscreen playback is required.
if (isVideo() && document()->page() && document()->page()->chrome()->requiresFullscreenForVideoPlayback())
return true;
return hasAttribute(controlsAttr);
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment