Selaa lähdekoodia

style: Renames variables, adds a FIXME.

master
Boris Grozev 8 vuotta sitten
vanhempi
commit
c0c198098b

+ 2
- 3
modules/UI/videolayout/RemoteVideo.js Näytä tiedosto

@@ -440,11 +440,10 @@ RemoteVideo.prototype.waitForPlayback = function (streamElement, stream) {
440 440
 
441 441
     var self = this;
442 442
 
443
-    // Register 'onplaying' listener to trigger 'videoactive' on VideoLayout
444
-    // when video playback starts
443
+    // Triggers when video playback starts
445 444
     var onPlayingHandler = function () {
446 445
         self.wasVideoPlayed = true;
447
-        self.VideoLayout.videoactive(streamElement, self.id);
446
+        self.VideoLayout.remoteVideoActive(streamElement, self.id);
448 447
         streamElement.onplaying = null;
449 448
         // Refresh to show the video
450 449
         self.updateView();

+ 4
- 3
modules/UI/videolayout/VideoLayout.js Näytä tiedosto

@@ -423,12 +423,13 @@ var VideoLayout = {
423 423
         remoteVideo.updateView();
424 424
     },
425 425
 
426
-    videoactive (videoelem, resourceJid) {
426
+    // FIXME: what does this do???
427
+    remoteVideoActive(videoElement, resourceJid) {
427 428
 
428
-        console.info(resourceJid + " video is now active", videoelem);
429
+        console.info(resourceJid + " video is now active", videoElement);
429 430
 
430 431
         VideoLayout.resizeThumbnails(
431
-            false, false, function() {$(videoelem).show();});
432
+            false, false, function() {$(videoElement).show();});
432 433
 
433 434
         // Update the large video to the last added video only if there's no
434 435
         // current dominant, focused speaker or update it to

Loading…
Peruuta
Tallenna