WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
Patch
(9.70 KB, patch)
2021-04-12 08:51 PDT
,
Philippe Normand
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
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
Created
attachment 425724
[details]
Patch
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
Created
attachment 425751
[details]
Patch
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
<
rdar://problem/76544696
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug