Title: [179280] trunk/Source/WebCore
Revision
179280
Author
[email protected]
Date
2015-01-28 13:01:06 -0800 (Wed, 28 Jan 2015)

Log Message

[WinCairo] Compile error in media player implementation.
https://bugs.webkit.org/show_bug.cgi?id=141004

Patch by [email protected] <[email protected]> on 2015-01-28
Reviewed by Brent Fulgham.

MediaPlayer::cachedResourceLoader() returns a pointer to the resource loader, not a reference.

* platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp:
(WebCore::MediaPlayerPrivateMediaFoundation::setSize):
(WebCore::MediaPlayerPrivateMediaFoundation::createVideoWindow):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (179279 => 179280)


--- trunk/Source/WebCore/ChangeLog	2015-01-28 20:48:19 UTC (rev 179279)
+++ trunk/Source/WebCore/ChangeLog	2015-01-28 21:01:06 UTC (rev 179280)
@@ -1,3 +1,16 @@
+2015-01-28  [email protected]  <[email protected]>
+
+        [WinCairo] Compile error in media player implementation.
+        https://bugs.webkit.org/show_bug.cgi?id=141004
+
+        Reviewed by Brent Fulgham.
+
+        MediaPlayer::cachedResourceLoader() returns a pointer to the resource loader, not a reference.
+
+        * platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp:
+        (WebCore::MediaPlayerPrivateMediaFoundation::setSize):
+        (WebCore::MediaPlayerPrivateMediaFoundation::createVideoWindow):
+
 2015-01-28  Jeremy Jones  <[email protected]>
 
         Prevent implicit animation when setting fullscreen background to clear.

Modified: trunk/Source/WebCore/platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp (179279 => 179280)


--- trunk/Source/WebCore/platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp	2015-01-28 20:48:19 UTC (rev 179279)
+++ trunk/Source/WebCore/platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp	2015-01-28 21:01:06 UTC (rev 179280)
@@ -196,8 +196,8 @@
     LayoutSize scrollOffset;
 
     FrameView* view = nullptr;
-    if (m_player && m_player->cachedResourceLoader() && m_player->cachedResourceLoader().document())
-        view = m_player->cachedResourceLoader().document()->view();
+    if (m_player && m_player->cachedResourceLoader() && m_player->cachedResourceLoader()->document())
+        view = m_player->cachedResourceLoader()->document()->view();
     if (view)
         scrollOffset = view->scrollOffsetForFixedPosition();
 
@@ -434,9 +434,9 @@
 {
     HWND hWndParent = nullptr;
     FrameView* view = nullptr;
-    if (!m_player || !m_player->cachedResourceLoader() || !m_player->cachedResourceLoader().document())
+    if (!m_player || !m_player->cachedResourceLoader() || !m_player->cachedResourceLoader()->document())
         return;
-    view = m_player->cachedResourceLoader().document()->view();
+    view = m_player->cachedResourceLoader()->document()->view();
     if (!view || !view->hostWindow())
         return;
     hWndParent = view->hostWindow()->platformPageClient();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to