RESOLVED FIXED 196606
[GStreamer][WebRTC] An audio track should be muted when capture is interrupted by the OS.
https://bugs.webkit.org/show_bug.cgi?id=196606
Summary [GStreamer][WebRTC] An audio track should be muted when capture is interrupte...
Michael Catanzaro
Reported 2019-04-04 08:59:25 PDT
An audio track should be muted when capture is interrupted by the OS, as implemented for Apple in r237837
Attachments
Patch (9.68 KB, patch)
2021-04-12 01:32 PDT, Philippe Normand
no flags
Patch (9.70 KB, patch)
2021-04-12 08:51 PDT, Philippe Normand
no flags
Philippe Normand
Comment 1 2021-04-11 09:14:20 PDT
*** Bug 222569 has been marked as a duplicate of this bug. ***
Philippe Normand
Comment 2 2021-04-12 01:32:07 PDT
Xabier Rodríguez Calvar
Comment 3 2021-04-12 03:56:36 PDT
Comment on attachment 425724 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=425724&action=review > Source/WebCore/platform/mediastream/gstreamer/GStreamerAudioCaptureSource.h:56 > + bool interrupted() const final; isInterrupted > Source/WebCore/platform/mediastream/gstreamer/GStreamerCapturer.cpp:188 > + gboolean drop; isDropping > Source/WebCore/platform/mediastream/gstreamer/GStreamerCapturer.cpp:193 > +void GStreamerCapturer::setInterrupted(bool interrupted) isInterrupted. > Source/WebCore/platform/mediastream/gstreamer/MockRealtimeAudioSourceGStreamer.cpp:143 > +void MockRealtimeAudioSourceGStreamer::setInterruptedForTesting(bool interrupted) isInterrupted > Source/WebCore/platform/mediastream/gstreamer/MockRealtimeAudioSourceGStreamer.h:48 > + bool interrupted() const final { return m_interrupted; }; isInterrupted > Source/WebCore/platform/mediastream/gstreamer/MockRealtimeAudioSourceGStreamer.h:56 > + bool m_interrupted { false }; m_isInterrupted
Philippe Normand
Comment 4 2021-04-12 08:50:47 PDT
Comment on attachment 425724 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=425724&action=review >> Source/WebCore/platform/mediastream/gstreamer/GStreamerAudioCaptureSource.h:56 >> + bool interrupted() const final; > > isInterrupted This is part of the RealtimeMediaSource interface, I'm not going to rename it in this patch :)
Philippe Normand
Comment 5 2021-04-12 08:51:08 PDT
EWS
Comment 6 2021-04-12 10:18:35 PDT
Committed r275827 (236397@main): <https://commits.webkit.org/236397@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 425751 [details].
Radar WebKit Bug Importer
Comment 7 2021-04-12 10:19:18 PDT
Note You need to log in before you can comment on or make changes to this bug.