Title: [277140] trunk/Source/WebCore
- Revision
- 277140
- Author
- [email protected]
- Date
- 2021-05-06 17:13:06 -0700 (Thu, 06 May 2021)
Log Message
r277122 caused a crash in GPU Process
https://bugs.webkit.org/show_bug.cgi?id=225492
<rdar://problem/77630141>
Reviewed by Jer Noble.
* platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::createAVPlayer): Have the time observer
call back on the main queue.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (277139 => 277140)
--- trunk/Source/WebCore/ChangeLog 2021-05-06 23:39:26 UTC (rev 277139)
+++ trunk/Source/WebCore/ChangeLog 2021-05-07 00:13:06 UTC (rev 277140)
@@ -1,3 +1,15 @@
+2021-05-06 Eric Carlson <[email protected]>
+
+ r277122 caused a crash in GPU Process
+ https://bugs.webkit.org/show_bug.cgi?id=225492
+ <rdar://problem/77630141>
+
+ Reviewed by Jer Noble.
+
+ * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
+ (WebCore::MediaPlayerPrivateAVFoundationObjC::createAVPlayer): Have the time observer
+ call back on the main queue.
+
2021-05-06 Devin Rousso <[email protected]>
Sampled Page Top Color: make hit tests consider elements with `pointer-events: none`
Modified: trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm (277139 => 277140)
--- trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm 2021-05-06 23:39:26 UTC (rev 277139)
+++ trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm 2021-05-07 00:13:06 UTC (rev 277140)
@@ -1104,7 +1104,7 @@
#endif
ASSERT(!m_currentTimeObserver);
- m_currentTimeObserver = [m_avPlayer addPeriodicTimeObserverForInterval:CMTimeMake(1, 10) queue:dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0) usingBlock:[this] (CMTime time) {
+ m_currentTimeObserver = [m_avPlayer addPeriodicTimeObserverForInterval:CMTimeMake(1, 10) queue:dispatch_get_main_queue() usingBlock:[this] (CMTime time) {
currentMediaTimeDidChange(PAL::toMediaTime(time));
}];
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
