Skip to content

[GStreamer][MediaStream] Fix time metadata handling in CanvasCaptureM…#1686

Merged
magomez merged 1 commit into
wpe-2.46from
pgorszkowski/2.46/fix_time_metadata_handling_in_CanvasCaptureMediaStreamTrack
Jun 11, 2026
Merged

[GStreamer][MediaStream] Fix time metadata handling in CanvasCaptureM…#1686
magomez merged 1 commit into
wpe-2.46from
pgorszkowski/2.46/fix_time_metadata_handling_in_CanvasCaptureMediaStreamTrack

Conversation

@pgorszkowski-igalia

@pgorszkowski-igalia pgorszkowski-igalia commented Jun 10, 2026

Copy link
Copy Markdown

…ediaStreamTrack

https://bugs.webkit.org/show_bug.cgi?id=295466

Reviewed by Xabier Rodriguez-Calvar.

Metadata is attached to the GStreamer buffer associated with each video frame. In case of canvas capture the captureTime was lost, because our implementation of videoFrameAvailable doesn't use it.

  • Source/WebCore/Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp: (WebCore::CanvasCaptureMediaStreamTrack::Source::captureCanvas):
  • Source/WebCore/platform/graphics/gstreamer/VideoFrameGStreamer.cpp: (WebCore::VideoFrameGStreamer::VideoFrameGStreamer): (WebCore::VideoFrameGStreamer::setMetadata):
  • Source/WebCore/platform/graphics/gstreamer/VideoFrameGStreamer.h:

Canonical link: https://commits.webkit.org/297043@main
d9a2faa

Build-Tests Layout-Tests
✅ 🛠 wpe-246-amd64-build ✅ 🧪 wpe-246-amd64-layout
✅ 🛠 wpe-246-arm32-build ✅ 🧪 wpe-246-arm32-layout

…ediaStreamTrack

https://bugs.webkit.org/show_bug.cgi?id=295466

Reviewed by Xabier Rodriguez-Calvar.

Metadata is attached to the GStreamer buffer associated with each video frame. In case of canvas
capture the captureTime was lost, because our implementation of videoFrameAvailable doesn't use
it.

* Source/WebCore/Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp:
(WebCore::CanvasCaptureMediaStreamTrack::Source::captureCanvas):
* Source/WebCore/platform/graphics/gstreamer/VideoFrameGStreamer.cpp:
(WebCore::VideoFrameGStreamer::VideoFrameGStreamer):
(WebCore::VideoFrameGStreamer::setMetadata):
* Source/WebCore/platform/graphics/gstreamer/VideoFrameGStreamer.h:

Canonical link: https://commits.webkit.org/297043@main
@magomez magomez merged commit 5702790 into wpe-2.46 Jun 11, 2026
@magomez magomez deleted the pgorszkowski/2.46/fix_time_metadata_handling_in_CanvasCaptureMediaStreamTrack branch June 11, 2026 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants