Title: [239663] trunk/Source/WebCore
- Revision
- 239663
- Author
- [email protected]
- Date
- 2019-01-06 09:15:29 -0800 (Sun, 06 Jan 2019)
Log Message
[WPE][GTK] Building with ENABLE_VIDEO=OFF fails trying to use Document MediaPlayback functions.
https://bugs.webkit.org/show_bug.cgi?id=193174
Patch by Pablo Saavedra <[email protected]> on 2019-01-06
Reviewed by Michael Catanzaro.
* page/Page.cpp:
(WebCore::Page::stopAllMediaPlayback):
(WebCore::Page::suspendAllMediaPlayback):
(WebCore::Page::resumeAllMediaPlayback):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (239662 => 239663)
--- trunk/Source/WebCore/ChangeLog 2019-01-06 05:05:09 UTC (rev 239662)
+++ trunk/Source/WebCore/ChangeLog 2019-01-06 17:15:29 UTC (rev 239663)
@@ -1,3 +1,15 @@
+2019-01-06 Pablo Saavedra <[email protected]>
+
+ [WPE][GTK] Building with ENABLE_VIDEO=OFF fails trying to use Document MediaPlayback functions.
+ https://bugs.webkit.org/show_bug.cgi?id=193174
+
+ Reviewed by Michael Catanzaro.
+
+ * page/Page.cpp:
+ (WebCore::Page::stopAllMediaPlayback):
+ (WebCore::Page::suspendAllMediaPlayback):
+ (WebCore::Page::resumeAllMediaPlayback):
+
2019-01-05 David Kilzer <[email protected]>
Leak of two CCRSACryptorRef (4.0 Kbytes/1 page each) in com.apple.WebKit.WebContent running WebKit layout tests
Modified: trunk/Source/WebCore/page/Page.cpp (239662 => 239663)
--- trunk/Source/WebCore/page/Page.cpp 2019-01-06 05:05:09 UTC (rev 239662)
+++ trunk/Source/WebCore/page/Page.cpp 2019-01-06 17:15:29 UTC (rev 239663)
@@ -1718,14 +1718,17 @@
void Page::stopAllMediaPlayback()
{
+#if ENABLE(VIDEO)
for (Frame* frame = &mainFrame(); frame; frame = frame->tree().traverseNext()) {
if (auto* document = frame->document())
document->stopAllMediaPlayback();
}
+#endif
}
void Page::suspendAllMediaPlayback()
{
+#if ENABLE(VIDEO)
ASSERT(!m_mediaPlaybackIsSuspended);
if (m_mediaPlaybackIsSuspended)
return;
@@ -1736,10 +1739,12 @@
}
m_mediaPlaybackIsSuspended = true;
+#endif
}
void Page::resumeAllMediaPlayback()
{
+#if ENABLE(VIDEO)
ASSERT(m_mediaPlaybackIsSuspended);
if (!m_mediaPlaybackIsSuspended)
return;
@@ -1749,6 +1754,7 @@
if (auto* document = frame->document())
document->resumeAllMediaPlayback();
}
+#endif
}
#if ENABLE(MEDIA_SESSION)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
