mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

Calling: Prevent video flicker when switching audio devices

Browse source
This commit is contained in:
Josh Perez 2020-08-31 19:24:47 -04:00 • committed by Josh Perez
parent fd06ac9d57
commit c2aa8eb82b
1 changed files with 14 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
ts/services/calling.ts
View file

@ -265,7 +265,8 @@ export class CallingClass {
this.lastMediaDeviceSettings,
newSettings
);
await this.selectPreferredMediaDevices(newSettings);
await this.selectPreferredDevices(newSettings);
this.lastMediaDeviceSettings = newSettings;
this.uxActions?.refreshIODevices(newSettings);
}
@ -407,9 +408,19 @@ export class CallingClass {
);
}
private async selectPreferredMediaDevices(
private async selectPreferredDevices(
settings: MediaDeviceSettings
): Promise<void> {
if (
(!this.lastMediaDeviceSettings && settings.selectedCamera) ||
(this.lastMediaDeviceSettings &&
settings.selectedCamera &&
this.lastMediaDeviceSettings.selectedCamera !== settings.selectedCamera)
) {
window.log.info('MediaDevice: selecting camera', settings.selectedCamera);
await this.videoCapturer.setPreferredDevice(settings.selectedCamera);
}
// Assume that the MediaDeviceSettings have been obtained very recently and the index is still valid (no devices have been plugged in in between).
if (settings.selectedMicrophone) {
window.log.info(
@ -418,6 +429,7 @@ export class CallingClass {
);
RingRTC.setAudioInput(settings.selectedMicrophone.index);
}
if (settings.selectedSpeaker) {
window.log.info(
'MediaDevice: selecting speaker',
@ -425,10 +437,6 @@ export class CallingClass {
);
RingRTC.setAudioOutput(settings.selectedSpeaker.index);
}
if (settings.selectedCamera) {
window.log.info('MediaDevice: selecting camera', settings.selectedCamera);
await this.videoCapturer.setPreferredDevice(settings.selectedCamera);
}
}
private async requestCameraPermissions(): Promise<boolean> {