From a38d6b4855e9a001b1d982e05a873269f93e2a2a Mon Sep 17 00:00:00 2001
From: pabloFuente <pablofuenteperez@gmail.com>
Date: Tue, 10 May 2022 15:45:43 +0200
Subject: [PATCH] openvidu-browser: remove unused parameter in
 Publisher#getVideoDimensions

---
 openvidu-browser/src/OpenVidu/OpenVidu.ts  | 2 +-
 openvidu-browser/src/OpenVidu/Publisher.ts | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/openvidu-browser/src/OpenVidu/OpenVidu.ts b/openvidu-browser/src/OpenVidu/OpenVidu.ts
index 16a6227c..624a0ebb 100644
--- a/openvidu-browser/src/OpenVidu/OpenVidu.ts
+++ b/openvidu-browser/src/OpenVidu/OpenVidu.ts
@@ -635,7 +635,7 @@ export class OpenVidu {
       if (attempts > MAX_ATTEMPTS) {
         clearTimeout(repeatUntilChangeOrMaxAttempts);
       }
-      publisher.getVideoDimensions(publisher.stream.getMediaStream()).then(newDimensions => {
+      publisher.getVideoDimensions().then(newDimensions => {
         if (newDimensions.width !== oldWidth || newDimensions.height !== oldHeight) {
           clearTimeout(repeatUntilChangeOrMaxAttempts);
           this.sendVideoDimensionsChangedEvent(publisher, reason, oldWidth, oldHeight, newDimensions.width, newDimensions.height);
diff --git a/openvidu-browser/src/OpenVidu/Publisher.ts b/openvidu-browser/src/OpenVidu/Publisher.ts
index 6a93a2eb..f1a09875 100644
--- a/openvidu-browser/src/OpenVidu/Publisher.ts
+++ b/openvidu-browser/src/OpenVidu/Publisher.ts
@@ -438,7 +438,7 @@ export class Publisher extends StreamManager {
 
                 if (this.stream.isSendVideo()) {
                     // Has video track
-                    this.getVideoDimensions(mediaStream).then(dimensions => {
+                    this.getVideoDimensions().then(dimensions => {
                         this.stream.videoDimensions = {
                             width: dimensions.width,
                             height: dimensions.height
@@ -670,7 +670,7 @@ export class Publisher extends StreamManager {
      * and then try to use MediaStreamTrack.getSettingsMethod(). If not available, then we
      * use the HTMLVideoElement properties videoWidth and videoHeight
      */
-    getVideoDimensions(mediaStream: MediaStream): Promise<{ width: number, height: number }> {
+    getVideoDimensions(): Promise<{ width: number, height: number }> {
         return new Promise((resolve, reject) => {
 
             // Ionic iOS and Safari iOS supposedly require the video element to actually exist inside the DOM
@@ -770,7 +770,7 @@ export class Publisher extends StreamManager {
         mediaStream.removeTrack(removedTrack);
         removedTrack.stop();
         mediaStream.addTrack(track);
-        if (track.kind === 'video' && this.stream.isLocalStreamPublished) {
+        if (track.kind === 'video' && this.stream.isLocalStreamPublished && updateLastConstraints) {
             this.openvidu.sendNewVideoDimensionsIfRequired(this, 'trackReplaced', 50, 30);
             this.session.sendVideoData(this.stream.streamManager, 5, true, 5);
         }