[webkit-changes] [WebKit/WebKit] 36e348: Picture-in-picture is broken if srcObject is video...

youennf noreply at github.com
Tue Oct 17 01:22:22 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 36e3488b27368e0924a1c439b0120ac64aa36bfe
      https://github.com/WebKit/WebKit/commit/36e3488b27368e0924a1c439b0120ac64aa36bfe
  Author: Youenn Fablet <youennf at gmail.com>
  Date:   2023-10-17 (Tue, 17 Oct 2023)

  Changed paths:
    M Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.h
    M Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.mm

  Log Message:
  -----------
  Picture-in-picture is broken if srcObject is video stream (Safari 17 on macOS 14)
https://bugs.webkit.org/show_bug.cgi?id=262479
rdar://116465668

Reviewed by Andy Estes.

We have to implement MediaPlayerPrivate::requestHostingContextID to ensure the full screen setup is done in a timely manner.
Otherwise, we do half of the pip procedure without terminating it.

Manullay tested with https://bug-262479-attachments.webkit.org/attachment.cgi?id=468022.

* Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.h:
* Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.mm:
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::layersAreInitialized):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::requestHostingContextID):

Canonical link: https://commits.webkit.org/269403@main




More information about the webkit-changes mailing list