Title: [282944] branches/safari-612-branch/Source/WebCore
- Revision
- 282944
- Author
- repst...@apple.com
- Date
- 2021-09-22 22:14:32 -0700 (Wed, 22 Sep 2021)
Log Message
Cherry-pick r282774. rdar://problem/83429893
[GPUP] Videos appear black but audio plays normally
https://bugs.webkit.org/show_bug.cgi?id=230471
<rdar://82121369>
Reviewed by Youenn Fablet.
For some videos, the media player (MediaPlayerPrivateAVFoundation) starts
with the "MediaRenderingToContext" mode, and it switches to the "MediaRenderingToLayer"
mode later. When the mode switch happens, we have to destroy the context
video renderer before creating the video layer. Otherwise, we won't receive
the notification regarding "readyForDisplay" key path, which drives the
`firstVideoFrameAvailable` callback.
`destroyContextVideoRenderer()` destroys the `m_videoOutput`, but it will
be recreated by `createVideoLayer()`.
Manually tested.
* platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:
(WebCore::MediaPlayerPrivateAVFoundation::setUpVideoRendering):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@282774 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-612-branch/Source/WebCore/ChangeLog (282943 => 282944)
--- branches/safari-612-branch/Source/WebCore/ChangeLog 2021-09-23 05:14:29 UTC (rev 282943)
+++ branches/safari-612-branch/Source/WebCore/ChangeLog 2021-09-23 05:14:32 UTC (rev 282944)
@@ -1,5 +1,56 @@
2021-09-22 Alan Coon <alanc...@apple.com>
+ Cherry-pick r282774. rdar://problem/83429893
+
+ [GPUP] Videos appear black but audio plays normally
+ https://bugs.webkit.org/show_bug.cgi?id=230471
+ <rdar://82121369>
+
+ Reviewed by Youenn Fablet.
+
+ For some videos, the media player (MediaPlayerPrivateAVFoundation) starts
+ with the "MediaRenderingToContext" mode, and it switches to the "MediaRenderingToLayer"
+ mode later. When the mode switch happens, we have to destroy the context
+ video renderer before creating the video layer. Otherwise, we won't receive
+ the notification regarding "readyForDisplay" key path, which drives the
+ `firstVideoFrameAvailable` callback.
+
+ `destroyContextVideoRenderer()` destroys the `m_videoOutput`, but it will
+ be recreated by `createVideoLayer()`.
+
+ Manually tested.
+
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:
+ (WebCore::MediaPlayerPrivateAVFoundation::setUpVideoRendering):
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@282774 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2021-09-20 Peng Liu <peng.l...@apple.com>
+
+ [GPUP] Videos appear black but audio plays normally
+ https://bugs.webkit.org/show_bug.cgi?id=230471
+ <rdar://82121369>
+
+ Reviewed by Youenn Fablet.
+
+ For some videos, the media player (MediaPlayerPrivateAVFoundation) starts
+ with the "MediaRenderingToContext" mode, and it switches to the "MediaRenderingToLayer"
+ mode later. When the mode switch happens, we have to destroy the context
+ video renderer before creating the video layer. Otherwise, we won't receive
+ the notification regarding "readyForDisplay" key path, which drives the
+ `firstVideoFrameAvailable` callback.
+
+ `destroyContextVideoRenderer()` destroys the `m_videoOutput`, but it will
+ be recreated by `createVideoLayer()`.
+
+ Manually tested.
+
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:
+ (WebCore::MediaPlayerPrivateAVFoundation::setUpVideoRendering):
+
+2021-09-22 Alan Coon <alanc...@apple.com>
+
Cherry-pick r282707. rdar://problem/83429953
[JSC] Add fast property enumeration mode for JSON.stringify
Modified: branches/safari-612-branch/Source/WebCore/platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp (282943 => 282944)
--- branches/safari-612-branch/Source/WebCore/platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp 2021-09-23 05:14:29 UTC (rev 282943)
+++ branches/safari-612-branch/Source/WebCore/platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp 2021-09-23 05:14:32 UTC (rev 282944)
@@ -136,6 +136,7 @@
break;
case MediaRenderingToLayer:
+ destroyContextVideoRenderer();
createVideoLayer();
break;
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes