Bläddra i källkod

ref(video-layout): remote thumbnail should not update large video directly

master
Leonard Kim 7 år sedan
förälder
incheckning
92414a346a
2 ändrade filer med 6 tillägg och 8 borttagningar
  1. 1
    8
      modules/UI/videolayout/RemoteVideo.js
  2. 5
    0
      modules/UI/videolayout/VideoLayout.js

+ 1
- 8
modules/UI/videolayout/RemoteVideo.js Visa fil

366
     logger.info(`${isVideo ? 'Video' : 'Audio'
366
     logger.info(`${isVideo ? 'Video' : 'Audio'
367
     } removed ${this.id}`, select);
367
     } removed ${this.id}`, select);
368
 
368
 
369
-    // when removing only the video element and we are on stage
370
-    // update the stage
371
-    if (isVideo && this.isCurrentlyOnLargeVideo()) {
372
-        this.VideoLayout.updateLargeVideo(this.id);
373
-    } else {
374
-        // Missing video stream will affect display mode
375
-        this.updateView();
376
-    }
369
+    this.updateView();
377
 };
370
 };
378
 
371
 
379
 /**
372
 /**

+ 5
- 0
modules/UI/videolayout/VideoLayout.js Visa fil

347
         if (remoteVideo) {
347
         if (remoteVideo) {
348
             remoteVideo.removeRemoteStreamElement(stream);
348
             remoteVideo.removeRemoteStreamElement(stream);
349
         }
349
         }
350
+
351
+        if (stream.isVideoTrack() && this.isCurrentlyOnLarge(id)) {
352
+            this.updateLargeVideo(id);
353
+        }
354
+
350
         this.updateMutedForNoTracks(id, stream.getType());
355
         this.updateMutedForNoTracks(id, stream.getType());
351
     },
356
     },
352
 
357
 

Laddar…
Avbryt
Spara