Title: [266552] branches/safari-610-branch/Source/WebCore
Revision
266552
Author
alanc...@apple.com
Date
2020-09-03 14:31:09 -0700 (Thu, 03 Sep 2020)

Log Message

Cherry-pick r266144. rdar://problem/68168942

    When using airplay with Youtube, the Youtube tab becomes completely empty and is unresponsive for an extended period of time if we switch the tab
    https://bugs.webkit.org/show_bug.cgi?id=215821

    Reviewed by Eric Carlson.

    We should ignore the request to paint the current video frame when we are using airplay.
    It is not necessary to do so, and -[AVAssetImageGenerator copyCGImageAtTime:actualTime:error:]
    will block the web process for a long time if the video is airplaying.

    * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
    (WebCore::MediaPlayerPrivateAVFoundationObjC::paintCurrentFrameInContext):

    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266144 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Modified Paths

Diff

Modified: branches/safari-610-branch/Source/WebCore/ChangeLog (266551 => 266552)


--- branches/safari-610-branch/Source/WebCore/ChangeLog	2020-09-03 21:31:06 UTC (rev 266551)
+++ branches/safari-610-branch/Source/WebCore/ChangeLog	2020-09-03 21:31:09 UTC (rev 266552)
@@ -1,5 +1,38 @@
 2020-09-03  Alan Coon  <alanc...@apple.com>
 
+        Cherry-pick r266144. rdar://problem/68168942
+
+    When using airplay with Youtube, the Youtube tab becomes completely empty and is unresponsive for an extended period of time if we switch the tab
+    https://bugs.webkit.org/show_bug.cgi?id=215821
+    
+    Reviewed by Eric Carlson.
+    
+    We should ignore the request to paint the current video frame when we are using airplay.
+    It is not necessary to do so, and -[AVAssetImageGenerator copyCGImageAtTime:actualTime:error:]
+    will block the web process for a long time if the video is airplaying.
+    
+    * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
+    (WebCore::MediaPlayerPrivateAVFoundationObjC::paintCurrentFrameInContext):
+    
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266144 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2020-08-25  Peng Liu  <peng.l...@apple.com>
+
+            When using airplay with Youtube, the Youtube tab becomes completely empty and is unresponsive for an extended period of time if we switch the tab
+            https://bugs.webkit.org/show_bug.cgi?id=215821
+
+            Reviewed by Eric Carlson.
+
+            We should ignore the request to paint the current video frame when we are using airplay.
+            It is not necessary to do so, and -[AVAssetImageGenerator copyCGImageAtTime:actualTime:error:]
+            will block the web process for a long time if the video is airplaying.
+
+            * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
+            (WebCore::MediaPlayerPrivateAVFoundationObjC::paintCurrentFrameInContext):
+
+2020-09-03  Alan Coon  <alanc...@apple.com>
+
         Cherry-pick r266158. rdar://problem/67812825
 
     Web Share API Level 2 functions even when its experimental feature flag is disabled

Modified: branches/safari-610-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm (266551 => 266552)


--- branches/safari-610-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm	2020-09-03 21:31:06 UTC (rev 266551)
+++ branches/safari-610-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm	2020-09-03 21:31:09 UTC (rev 266552)
@@ -1523,7 +1523,7 @@
 
 void MediaPlayerPrivateAVFoundationObjC::paintCurrentFrameInContext(GraphicsContext& context, const FloatRect& rect)
 {
-    if (!metaDataAvailable() || context.paintingDisabled())
+    if (!metaDataAvailable() || context.paintingDisabled() || isCurrentPlaybackTargetWireless())
         return;
 
     setDelayCallbacks(true);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to