Changeset 82991 in webkit

Timestamp:

Apr 5, 2011 3:23:18 PM (13 years ago)

Author:

jer.noble@apple.com

Message:

2011-04-05 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

HTML5 Player Has no Fullscreen Mode
​https://bugs.webkit.org/show_bug.cgi?id=57795

• html/HTMLVideoElement.cpp: (WebCore::HTMLVideoElement::supportsFullscreen): Call the new ChromeClient API

supportsFullScreenForElement() before the old supportsFullScreenForNode() API.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/HTMLVideoElement.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-04-05  Jer Noble  <jer.noble@apple.com>
+
+        Reviewed by Darin Adler.
+
+        HTML5 Player Has no Fullscreen Mode
+        https://bugs.webkit.org/show_bug.cgi?id=57795
+
+        * html/HTMLVideoElement.cpp:
+        (WebCore::HTMLVideoElement::supportsFullscreen): Call the new ChromeClient API
+            supportsFullScreenForElement() before the old supportsFullScreenForNode() API.
+
 2011-04-05  Vsevolod Vlasov  <vsevik@chromium.org>
 

trunk/Source/WebCore/html/HTMLVideoElement.cpp

@@ -130 +130 @@
 
     // Check with the platform client.
+#if ENABLE(FULLSCREEN_API)
+    if (page->chrome()->client()->supportsFullScreenForElement(this, false))
+        return true;
+#endif
+
     return page->chrome()->client()->supportsFullscreenForNode(this);
 }