ref(qualitycontrol) Make ReceiveVideoController and SendVideoController default classes.

JitsiConference.js

@@ -32,8 +32,8 @@ import { E2EEncryption } from './modules/e2ee/E2EEncryption';
 import E2ePing from './modules/e2eping/e2eping';
 import Jvb121EventGenerator from './modules/event/Jvb121EventGenerator';
 import FeatureFlags from './modules/flags/FeatureFlags';
-import { ReceiveVideoController } from './modules/qualitycontrol/ReceiveVideoController';
-import { SendVideoController } from './modules/qualitycontrol/SendVideoController';
+import ReceiveVideoController from './modules/qualitycontrol/ReceiveVideoController';
+import SendVideoController from './modules/qualitycontrol/SendVideoController';
 import RecordingManager from './modules/recording/RecordingManager';
 import Settings from './modules/settings/Settings';
 import AudioOutputProblemDetector from './modules/statistics/AudioOutputProblemDetector';

modules/qualitycontrol/ReceiveVideoController.js

@@ -13,7 +13,7 @@ const LASTN_UNLIMITED = -1;
  * This class translates the legacy signaling format between the client and the bridge (that affects bandwidth
  * allocation) to the new format described here https://github.com/jitsi/jitsi-videobridge/blob/master/doc/allocation.md
  */
-export class ReceiverVideoConstraints {
+class ReceiverVideoConstraints {
     /**
      * Creates a new instance.
      */
@@ -159,7 +159,7 @@ export class ReceiverVideoConstraints {
  * determined by the application based on how the remote video streams need to be displayed. This class is responsible
  * for communicating these constraints to the bridge over the bridge channel.
  */
-export class ReceiveVideoController {
+export default class ReceiveVideoController {
     /**
      * Creates a new instance for a given conference.
      *

modules/qualitycontrol/ReceiveVideoController.spec.js

@@ -1,7 +1,7 @@
 import FeatureFlags from '../flags/FeatureFlags';
 import Listenable from '../util/Listenable';
 
-import { ReceiveVideoController } from './ReceiveVideoController';
+import ReceiveVideoController from './ReceiveVideoController';
 
 // JSDocs disabled for Mock classes to avoid duplication - check on the original classes for info.
 /* eslint-disable require-jsdoc */

modules/qualitycontrol/SendVideoController.js

@@ -9,7 +9,7 @@ import MediaSessionEvents from '../xmpp/MediaSessionEvents';
  * because local tracks are shared and while JVB may have no preference, the remote p2p may have and they may be totally
  * different.
  */
-export class SendVideoController {
+export default class SendVideoController {
     /**
      * Creates new instance for a given conference.
      *

modules/qualitycontrol/SendVideoController.spec.js

@@ -3,7 +3,7 @@ import RTCEvents from '../../service/RTC/RTCEvents';
 import Listenable from '../util/Listenable';
 import MediaSessionEvents from '../xmpp/MediaSessionEvents';
 
-import { SendVideoController } from './SendVideoController';
+import SendVideoController from './SendVideoController';
 
 // JSDocs disabled for Mock classes to avoid duplication - check on the original classes for info.
 /* eslint-disable require-jsdoc */

types/auto/JitsiConference.d.ts

@@ -1206,8 +1206,8 @@ import { E2EEncryption } from "./modules/e2ee/E2EEncryption";
 import { CodecSelection } from "./modules/RTC/CodecSelection";
 import E2ePing from "./modules/e2eping/e2eping";
 import RTC from "./modules/RTC/RTC";
-import { ReceiveVideoController } from "./modules/qualitycontrol/ReceiveVideoController";
-import { SendVideoController } from "./modules/qualitycontrol/SendVideoController";
+import ReceiveVideoController from "./modules/qualitycontrol/ReceiveVideoController";
+import SendVideoController from "./modules/qualitycontrol/SendVideoController";
 import ParticipantConnectionStatusHandler from "./modules/connectivity/ParticipantConnectionStatus";
 import Statistics from "./modules/statistics/statistics";
 import VADAudioAnalyser from "./modules/detection/VADAudioAnalyser";

types/auto/modules/qualitycontrol/ReceiveVideoController.d.ts

@@ -1,67 +1,9 @@
-/**
- * This class translates the legacy signaling format between the client and the bridge (that affects bandwidth
- * allocation) to the new format described here https://github.com/jitsi/jitsi-videobridge/blob/master/doc/allocation.md
- */
-export class ReceiverVideoConstraints {
-    _defaultConstraints: {
-        maxHeight: number;
-    };
-    _lastN: number;
-    _maxFrameHeight: number;
-    _selectedEndpoints: any[];
-    _receiverVideoConstraints: {
-        constraints: {};
-        defaultConstraints: any;
-        lastN: number;
-        onStageEndpoints: any[];
-        selectedEndpoints: any[];
-    };
-    /**
-     * Returns the receiver video constraints that need to be sent on the bridge channel.
-     */
-    get constraints(): {
-        constraints: {};
-        defaultConstraints: any;
-        lastN: number;
-        onStageEndpoints: any[];
-        selectedEndpoints: any[];
-    };
-    /**
-     * Updates the lastN field of the ReceiverVideoConstraints sent to the bridge.
-     *
-     * @param {number} value
-     * @returns {boolean} Returns true if the the value has been updated, false otherwise.
-     */
-    updateLastN(value: number): boolean;
-    /**
-     * Updates the resolution (height requested) in the contraints field of the ReceiverVideoConstraints
-     * sent to the bridge.
-     *
-     * @param {number} maxFrameHeight
-     * @requires {boolean} Returns true if the the value has been updated, false otherwise.
-     */
-    updateReceiveResolution(maxFrameHeight: number): boolean;
-    /**
-     * Updates the receiver constraints sent to the bridge.
-     *
-     * @param {Object} videoConstraints
-     * @returns {boolean} Returns true if the the value has been updated, false otherwise.
-     */
-    updateReceiverVideoConstraints(videoConstraints: any): boolean;
-    /**
-     * Updates the list of selected endpoints.
-     *
-     * @param {Array<string>} ids
-     * @returns {void}
-     */
-    updateSelectedEndpoints(ids: Array<string>): void;
-}
 /**
  * This class manages the receive video contraints for a given {@link JitsiConference}. These constraints are
  * determined by the application based on how the remote video streams need to be displayed. This class is responsible
  * for communicating these constraints to the bridge over the bridge channel.
  */
-export class ReceiveVideoController {
+export default class ReceiveVideoController {
     /**
      * Creates a new instance for a given conference.
      *
@@ -121,3 +63,62 @@ export class ReceiveVideoController {
      */
     setReceiverConstraints(constraints: any): void;
 }
+/**
+ * This class translates the legacy signaling format between the client and the bridge (that affects bandwidth
+ * allocation) to the new format described here https://github.com/jitsi/jitsi-videobridge/blob/master/doc/allocation.md
+ */
+declare class ReceiverVideoConstraints {
+    _defaultConstraints: {
+        maxHeight: number;
+    };
+    _lastN: number;
+    _maxFrameHeight: number;
+    _selectedEndpoints: any[];
+    _receiverVideoConstraints: {
+        constraints: {};
+        defaultConstraints: any;
+        lastN: number;
+        onStageEndpoints: any[];
+        selectedEndpoints: any[];
+    };
+    /**
+     * Returns the receiver video constraints that need to be sent on the bridge channel.
+     */
+    get constraints(): {
+        constraints: {};
+        defaultConstraints: any;
+        lastN: number;
+        onStageEndpoints: any[];
+        selectedEndpoints: any[];
+    };
+    /**
+     * Updates the lastN field of the ReceiverVideoConstraints sent to the bridge.
+     *
+     * @param {number} value
+     * @returns {boolean} Returns true if the the value has been updated, false otherwise.
+     */
+    updateLastN(value: number): boolean;
+    /**
+     * Updates the resolution (height requested) in the contraints field of the ReceiverVideoConstraints
+     * sent to the bridge.
+     *
+     * @param {number} maxFrameHeight
+     * @requires {boolean} Returns true if the the value has been updated, false otherwise.
+     */
+    updateReceiveResolution(maxFrameHeight: number): boolean;
+    /**
+     * Updates the receiver constraints sent to the bridge.
+     *
+     * @param {Object} videoConstraints
+     * @returns {boolean} Returns true if the the value has been updated, false otherwise.
+     */
+    updateReceiverVideoConstraints(videoConstraints: any): boolean;
+    /**
+     * Updates the list of selected endpoints.
+     *
+     * @param {Array<string>} ids
+     * @returns {void}
+     */
+    updateSelectedEndpoints(ids: Array<string>): void;
+}
+export {};

types/auto/modules/qualitycontrol/SendVideoController.d.ts

@@ -5,7 +5,7 @@
  * because local tracks are shared and while JVB may have no preference, the remote p2p may have and they may be totally
  * different.
  */
-export class SendVideoController {
+export default class SendVideoController {
     /**
      * Creates new instance for a given conference.
      *