ref(CustomSignaling): Remove ssrcOwners&chatRoom.

jitsi · Oct 6, 2023 · fb61f63 · fb61f63
1 parent 3d55666
commit fb61f63
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 80 deletions.
diff --git a/modules/proxyconnection/CustomSignalingLayer.js b/modules/proxyconnection/CustomSignalingLayer.js
@@ -1,31 +1,9 @@
-import { getLogger } from '@jitsi/logger';
-
 import SignalingLayer from '../../service/RTC/SignalingLayer';
 
-const logger = getLogger(__filename);
-
 /**
  * Custom semi-mock implementation for the Proxy connection service.
  */
 export default class CustomSignalingLayer extends SignalingLayer {
-    /**
-     * Creates new instance.
-     */
-    constructor() {
-        super();
-
-        /**
-         * A map that stores SSRCs of remote streams.
-         * @type {Map<number, string>} maps SSRC number to jid
-         */
-        this.ssrcOwners = new Map();
-
-        /**
-         *
-         * @type {ChatRoom|null}
-         */
-        this.chatRoom = null;
-    }
 
     /**
      * @inheritDoc
@@ -44,8 +22,8 @@ export default class CustomSignalingLayer extends SignalingLayer {
     /**
      * @inheritDoc
      */
-    getSSRCOwner(ssrc) {
-        return this.ssrcOwners.get(ssrc);
+    getSSRCOwner() {
+        return undefined;
     }
 
     /**
@@ -55,45 +33,6 @@ export default class CustomSignalingLayer extends SignalingLayer {
         return undefined;
     }
 
-    /**
-     * @inheritDoc
-     */
-    removeSSRCOwners(ssrcList) {
-        if (!ssrcList?.length) {
-            return;
-        }
-
-        for (const ssrc of ssrcList) {
-            this.ssrcOwners.delete(ssrc);
-        }
-    }
-
-    /**
-     * Sets the <tt>ChatRoom</tt> instance used.
-     * @param {ChatRoom} room
-     */
-    setChatRoom(room) {
-        this.chatRoom = room;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    setSSRCOwner(ssrc, endpointId) {
-        if (typeof ssrc !== 'number') {
-            throw new TypeError(`SSRC(${ssrc}) must be a number`);
-        }
-
-        // Now signaling layer instance is shared between different JingleSessionPC instances, so although very unlikely
-        // an SSRC conflict could potentially occur. Log a message to make debugging easier.
-        const existingOwner = this.ssrcOwners.get(ssrc);
-
-        if (existingOwner && existingOwner !== endpointId) {
-            logger.error(`SSRC owner re-assigned from ${existingOwner} to ${endpointId}`);
-        }
-        this.ssrcOwners.set(ssrc, endpointId);
-    }
-
     /**
      * @inheritDoc
      */
@@ -107,19 +46,4 @@ export default class CustomSignalingLayer extends SignalingLayer {
     setTrackVideoType(sourceName, videoType) { // eslint-disable-line no-unused-vars
         return false;
     }
-
-    /**
-     * @inheritDoc
-     */
-    updateSsrcOwnersOnLeave(id) {
-        const ssrcs = Array.from(this.ssrcOwners)
-            .filter(entry => entry[1] === id)
-            .map(entry => entry[0]);
-
-        if (!ssrcs?.length) {
-            return;
-        }
-
-        this.removeSSRCOwners(ssrcs);
-    }
 }
diff --git a/modules/proxyconnection/ProxyConnectionPC.js b/modules/proxyconnection/ProxyConnectionPC.js
@@ -268,8 +268,6 @@ export default class ProxyConnectionPC {
 
         const signalingLayer = new CustomSignalingLayer();
 
-        signalingLayer.setChatRoom(roomStub);
-
         /**
          * An additional initialize call is necessary to properly set instance
          * variable for calling.