Merge pull request #365 from saghul/disableRtx-default

rtx: set disableRtx default to true

modules/RTC/TraceablePeerConnection.js

@@ -894,7 +894,7 @@ TraceablePeerConnection.prototype.generateNewStreamSSRCInfo = function () {
     } else {
         ssrcInfo = {ssrcs: [SDPUtil.generateSsrc()], groups: []};
     }
-    if (!this.options.disableRtx) {
+    if (!this.options.disableRtx && !RTCBrowserType.isFirefox()) {
         // Specifically use a for loop here because we'll
         //  be adding to the list we're iterating over, so we
         //  only want to iterate through the items originally

modules/xmpp/moderator.js

@@ -147,13 +147,11 @@ Moderator.prototype.createConferenceIq =  function () {
                 value: this.options.conference.channelLastN
             }).up();
     }
-    if (this.options.conference.disableRtx !== undefined) {
-        elem.c(
-            'property', {
-                name: 'disableRtx',
-                value: this.options.conference.disableRtx
-            }).up();
-    }
+    elem.c(
+        'property', {
+            name: 'disableRtx',
+            value: !!this.options.conference.disableRtx
+        }).up();
     elem.c(
         'property', {
             name: 'enableLipSync',

modules/xmpp/xmpp.js

@@ -66,8 +66,7 @@ export default class XMPP extends Listenable {
         this.caps.addFeature('urn:xmpp:jingle:apps:rtp:audio');
         this.caps.addFeature('urn:xmpp:jingle:apps:rtp:video');
 
-        if (RTCBrowserType.isChrome() || RTCBrowserType.isOpera()
-            || RTCBrowserType.isTemasysPluginUsed()) {
+        if (!this.options.disableRtx && !RTCBrowserType.isFirefox()) {
             this.caps.addFeature('urn:ietf:rfc:4588');
         }