[webkit-changes] [WebKit/WebKit] 80d318: MediaStream from canvas.captureStream and then rea...

[youennf]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 80d318b89f6fc9962c63e4bbaa2947ca3c962b89
      https://github.com/WebKit/WebKit/commit/80d318b89f6fc9962c63e4bbaa2947ca3c962b89
  Author: Youenn Fablet <youenn at apple.com>
  Date:   2025-02-28 (Fri, 28 Feb 2025)

  Changed paths:
    M LayoutTests/http/wpt/mediastream/transfer-mediastreamtrack-to-worker-expected.txt
    M LayoutTests/http/wpt/mediastream/transfer-mediastreamtrack-to-worker.html
    M Source/WebCore/Modules/mediastream/MediaStreamTrackProcessor.cpp
    M Source/WebCore/Modules/webcodecs/WebCodecsVideoFrame.h
    M Source/WebCore/platform/VideoPixelFormat.h

  Log Message:
  -----------
  MediaStream from canvas.captureStream and then read VideoFrame using MediaStreamTrackProcessor, copyTo does not produce any content.
rdar://145454281
https://bugs.webkit.org/show_bug.cgi?id=287787

Reviewed by Eric Carlson.

We were setting the pixel format to I420, which is not true for canvas video frame.
Instead, we now use the underlying pixel format for WebCodecsVideoFrame format.

Covered by added test.

* LayoutTests/http/wpt/mediastream/transfer-mediastreamtrack-to-worker-expected.txt:
* LayoutTests/http/wpt/mediastream/transfer-mediastreamtrack-to-worker.html:
* Source/WebCore/Modules/mediastream/MediaStreamTrackProcessor.cpp:
(WebCore::MediaStreamTrackProcessor::VideoFrameObserver::takeVideoFrame):
* Source/WebCore/Modules/webcodecs/WebCodecsVideoFrame.h:
* Source/WebCore/platform/VideoPixelFormat.h:

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



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications