Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 8ce9ffa7193733b471ead3e2ec7b67bfaaccf9c7
https://github.com/WebKit/WebKit/commit/8ce9ffa7193733b471ead3e2ec7b67bfaaccf9c7
Author: Xabier Rodriguez-Calvar <[email protected]>
Date: 2024-02-26 (Mon, 26 Feb 2024)
Changed paths:
M Source/WebCore/platform/graphics/gstreamer/GStreamerCommon.h
M Source/WebCore/platform/graphics/gstreamer/VideoFrameGStreamer.cpp
M
Source/WebCore/platform/mediastream/libwebrtc/gstreamer/GStreamerVideoFrameLibWebRTC.cpp
Log Message:
-----------
[GStreamer] Make GstMappedFrame assert when it is not initialized
https://bugs.webkit.org/show_bug.cgi?id=269980
Reviewed by Carlos Garcia Campos.
This way we can align the implementation of GstMappedFrame with GstMappedBuffer
and assert when its data is accessed but
it was not properly mapped at initialization.
Even when now it is properly protected at members access, additional checks
were added in some places where its
instantiation was not being checked.
A fly-by improvement is making the constructor taking GRefPtr<GstSample> to
take const & to avoid unnecessary ref/unref.
* Source/WebCore/platform/graphics/gstreamer/GStreamerCommon.h:
(WebCore::GstMappedFrame::GstMappedFrame):
(WebCore::GstMappedFrame::get):
(WebCore::GstMappedFrame::ComponentData const):
(WebCore::GstMappedFrame::ComponentStride const):
(WebCore::GstMappedFrame::info):
(WebCore::GstMappedFrame::width const):
(WebCore::GstMappedFrame::height const):
(WebCore::GstMappedFrame::format const):
(WebCore::GstMappedFrame::planeData const):
(WebCore::GstMappedFrame::planeStride const):
(WebCore::GstMappedFrame::isValid const):
(WebCore::GstMappedFrame::operator! const):
* Source/WebCore/platform/graphics/gstreamer/VideoFrameGStreamer.cpp:
(WebCore::VideoFrame::copyTo):
*
Source/WebCore/platform/mediastream/libwebrtc/gstreamer/GStreamerVideoFrameLibWebRTC.cpp:
(WebCore::GStreamerVideoFrameLibWebRTC::ToI420):
Canonical link: https://commits.webkit.org/275320@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
