Skip to content

Commit 23a801c

Browse files
philncalvaris
authored andcommitted
[GStreamer] Audio sinks created by media players leak
https://bugs.webkit.org/show_bug.cgi?id=256899 Reviewed by Xabier Rodriguez-Calvar. Replace the leaky g_object_get() with a simple accessor to the m_audioSink member variable. * Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivateGStreamer::audioSink const): Deleted. * Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h: (WebCore::MediaPlayerPrivateGStreamer::audioSink const): Canonical link: https://commits.webkit.org/264193@main
1 parent cf37b74 commit 23a801c

2 files changed

Lines changed: 1 addition & 11 deletions

File tree

Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1339,16 +1339,6 @@ GstElement* MediaPlayerPrivateGStreamer::createAudioSink()
13391339
#endif
13401340
}
13411341

1342-
GstElement* MediaPlayerPrivateGStreamer::audioSink() const
1343-
{
1344-
if (!m_pipeline)
1345-
return nullptr;
1346-
1347-
GstElement* sink;
1348-
g_object_get(m_pipeline.get(), "audio-sink", &sink, nullptr);
1349-
return sink;
1350-
}
1351-
13521342
bool MediaPlayerPrivateGStreamer::isMediaStreamPlayer() const
13531343
{
13541344
#if ENABLE(MEDIA_STREAM)

Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -498,7 +498,7 @@ class MediaPlayerPrivateGStreamer : public MediaPlayerPrivateInterface
498498

499499
GstElement* createVideoSink();
500500
GstElement* createAudioSink();
501-
GstElement* audioSink() const;
501+
GstElement* audioSink() const { return m_audioSink.get(); }
502502

503503
bool isMediaStreamPlayer() const;
504504

0 commit comments

Comments
 (0)