Browse Source

Uses 'null' in JitsiTrack.detach, but sets empty string in Chrome implementation when the src is not specified.

master
paweldomas 9 years ago
parent
commit
0c33f7c53a
2 changed files with 5 additions and 2 deletions
  1. 2
    2
      modules/RTC/JitsiTrack.js
  2. 3
    0
      modules/RTC/RTCUtils.js

+ 2
- 2
modules/RTC/JitsiTrack.js View File

@@ -192,7 +192,7 @@ JitsiTrack.prototype.detach = function (container) {
192 192
     {
193 193
         if(!container)
194 194
         {
195
-            require("./RTCUtils").setVideoSrc(this.containers[i], '');
195
+            require("./RTCUtils").setVideoSrc(this.containers[i], null);
196 196
         }
197 197
         if(!container || $(this.containers[i]).is($(container)))
198 198
         {
@@ -201,7 +201,7 @@ JitsiTrack.prototype.detach = function (container) {
201 201
     }
202 202
 
203 203
     if(container) {
204
-        require("./RTCUtils").setVideoSrc(container, '');
204
+        require("./RTCUtils").setVideoSrc(container, null);
205 205
     }
206 206
 }
207 207
 

+ 3
- 0
modules/RTC/RTCUtils.js View File

@@ -505,6 +505,9 @@ var RTCUtils = {
505 505
                     return element.getAttribute("src");
506 506
                 };
507 507
                 this.setVideoSrc = function (element, src) {
508
+                    if (!src) {
509
+                        src = '';
510
+                    }
508 511
                     if (element)
509 512
                         element.setAttribute("src", src);
510 513
                 };

Loading…
Cancel
Save