Title: [120213] trunk/Source/WebKit/chromium
Revision
120213
Author
[email protected]
Date
2012-06-13 08:29:34 -0700 (Wed, 13 Jun 2012)

Log Message

[Chromium] Fix Clang build with USE(NATIVE_FULLSCREEN_VIDEO)=1
https://bugs.webkit.org/show_bug.cgi?id=88987

Reviewed by Simon Hausmann.

The enterFullscreen method is declared as const by MediaPlayerPrivateInterface,
so this one should be declared as such as well.


* src/WebMediaPlayerClientImpl.cpp:
(WebKit::WebMediaPlayerClientImpl::enterFullscreen):
* src/WebMediaPlayerClientImpl.h:
(WebMediaPlayerClientImpl):

Modified Paths

Diff

Modified: trunk/Source/WebKit/chromium/ChangeLog (120212 => 120213)


--- trunk/Source/WebKit/chromium/ChangeLog	2012-06-13 15:15:44 UTC (rev 120212)
+++ trunk/Source/WebKit/chromium/ChangeLog	2012-06-13 15:29:34 UTC (rev 120213)
@@ -1,3 +1,20 @@
+2012-06-13  Peter Beverloo  <[email protected]>
+
+        [Chromium] Fix Clang build with USE(NATIVE_FULLSCREEN_VIDEO)=1
+        https://bugs.webkit.org/show_bug.cgi?id=88987
+
+        Reviewed by Simon Hausmann.
+
+        The enterFullscreen method is declared as const by MediaPlayerPrivateInterface,
+        so this one should be declared as such as well.
+
+        Patch by Evgeniy Stepanov <[email protected]>
+
+        * src/WebMediaPlayerClientImpl.cpp:
+        (WebKit::WebMediaPlayerClientImpl::enterFullscreen):
+        * src/WebMediaPlayerClientImpl.h:
+        (WebMediaPlayerClientImpl):
+
 2012-06-13  MORITA Hajime  <[email protected]>
 
         REGRESSION(r118098): <content> element does not render distributed children when cloned from another document

Modified: trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp (120212 => 120213)


--- trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp	2012-06-13 15:15:44 UTC (rev 120212)
+++ trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp	2012-06-13 15:29:34 UTC (rev 120213)
@@ -370,7 +370,7 @@
 }
 
 #if USE(NATIVE_FULLSCREEN_VIDEO)
-void WebMediaPlayerClientImpl::enterFullscreen()
+void WebMediaPlayerClientImpl::enterFullscreen() const
 {
     if (m_webMediaPlayer)
         m_webMediaPlayer->enterFullscreen();

Modified: trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.h (120212 => 120213)


--- trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.h	2012-06-13 15:15:44 UTC (rev 120212)
+++ trunk/Source/WebKit/chromium/src/WebMediaPlayerClientImpl.h	2012-06-13 15:29:34 UTC (rev 120213)
@@ -139,7 +139,7 @@
     virtual unsigned audioDecodedByteCount() const;
     virtual unsigned videoDecodedByteCount() const;
 #if USE(NATIVE_FULLSCREEN_VIDEO)
-    virtual void enterFullscreen();
+    virtual void enterFullscreen() const;
     virtual void exitFullscreen();
     virtual bool canEnterFullscreen() const;
 #endif
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to