ambiser
/
openvidu
mirror of https://github.com/OpenVidu/openvidu.git
8
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

openvidu-browser: remove code smell in OpenVidu.ts

pull/803/head
pabloFuente 2023-05-05 13:20:33 +02:00
parent b4b808e3f1
commit 423e0d302f
1 changed files with 89 additions and 92 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
openvidu-browser/src/OpenVidu/OpenVidu.ts
View File

@ -512,15 +512,14 @@ export class OpenVidu {
* });
* ```
*/
getUserMedia(options: PublisherProperties): Promise<MediaStream> {
return new Promise<MediaStream>(async (resolve, reject) => {
async getUserMedia(options: PublisherProperties): Promise<MediaStream> {
const askForAudioStreamOnly = async (previousMediaStream: MediaStream, constraints: MediaStreamConstraints) => {
const definedAudioConstraint = constraints.audio === undefined ? true : constraints.audio;
const constraintsAux: MediaStreamConstraints = { audio: definedAudioConstraint, video: false };
try {
const audioOnlyStream = await navigator.mediaDevices.getUserMedia(constraintsAux);
previousMediaStream.addTrack(audioOnlyStream.getAudioTracks()[0]);
return resolve(previousMediaStream);
return previousMediaStream;
} catch (error) {
previousMediaStream.getAudioTracks().forEach((track) => {
track.stop();
@ -528,7 +527,7 @@ export class OpenVidu {
previousMediaStream.getVideoTracks().forEach((track) => {
track.stop();
});
return reject(this.generateAudioDeviceError(error, constraintsAux));
throw this.generateAudioDeviceError(error, constraintsAux);
}
};
@ -540,7 +539,7 @@ export class OpenVidu {
(!!myConstraints.videoTrack && myConstraints.constraints?.audio === false)
) {
// No need to call getUserMedia at all. Both tracks provided, or only AUDIO track provided or only VIDEO track provided
return resolve(this.addAlreadyProvidedTracks(myConstraints, new MediaStream()));
return this.addAlreadyProvidedTracks(myConstraints, new MediaStream());
} else {
// getUserMedia must be called. AUDIO or VIDEO are requesting a new track
@ -569,15 +568,14 @@ export class OpenVidu {
const mediaStream = await navigator.mediaDevices['getDisplayMedia']({ video: true });
this.addAlreadyProvidedTracks(myConstraints, mediaStream);
if (mustAskForAudioTrackLater) {
await askForAudioStreamOnly(mediaStream, <MediaStreamConstraints>myConstraints.constraints);
return;
return await askForAudioStreamOnly(mediaStream, <MediaStreamConstraints>myConstraints.constraints);
} else {
return resolve(mediaStream);
return mediaStream;
}
} catch (error) {
let errorName: OpenViduErrorName = OpenViduErrorName.SCREEN_CAPTURE_DENIED;
const errorMessage = error.toString();
return reject(new OpenViduError(errorName, errorMessage));
throw new OpenViduError(errorName, errorMessage);
}
} else {
// getDisplayMedia NOT supported. Can perform getUserMedia below with already calculated constraints
@ -594,9 +592,9 @@ export class OpenVidu {
const mediaStream = await navigator.mediaDevices.getUserMedia(constraintsAux);
this.addAlreadyProvidedTracks(myConstraints, mediaStream);
if (mustAskForAudioTrackLater) {
await askForAudioStreamOnly(mediaStream, <MediaStreamConstraints>myConstraints.constraints);
return await askForAudioStreamOnly(mediaStream, <MediaStreamConstraints>myConstraints.constraints);
} else {
return resolve(mediaStream);
return mediaStream;
}
} catch (error) {
let errorName: OpenViduErrorName;
@ -606,13 +604,12 @@ export class OpenVidu {
} else {
errorName = OpenViduErrorName.SCREEN_CAPTURE_DENIED;
}
return reject(new OpenViduError(errorName, errorMessage));
throw new OpenViduError(errorName, errorMessage);
}
}
} catch (error) {
reject(error);
throw error;
}
});
}
/* tslint:disable:no-empty */
@ -990,7 +987,7 @@ export class OpenVidu {
/**
* @hidden
*/
addAlreadyProvidedTracks(myConstraints: CustomMediaStreamConstraints, mediaStream: MediaStream, stream?: Stream) {
addAlreadyProvidedTracks(myConstraints: CustomMediaStreamConstraints, mediaStream: MediaStream, stream?: Stream): MediaStream {
if (!!myConstraints.videoTrack) {
mediaStream.addTrack(myConstraints.videoTrack);
if (!!stream) {