style: LD-6968 do changes after review

Author: vlad-livedigital

README.md

@@ -58,7 +58,7 @@ import WebRTCIssueDetector, {
   NetworkMediaSyncIssueDetector,
   AvailableOutgoingBitrateIssueDetector,
   UnknownVideoDecoderImplementationDetector,
-  DeadVideoTrackDetector,
+  FrozenVideoTrackDetector,
 } from 'webrtc-issue-detector';
 
 const widWithDefaultConstructorArgs = new WebRTCIssueDetector();
@@ -75,7 +75,7 @@ const widWithCustomConstructorArgs = new WebRTCIssueDetector({
     new NetworkMediaSyncIssueDetector(),
     new AvailableOutgoingBitrateIssueDetector(),
     new UnknownVideoDecoderImplementationDetector(),
-    new DeadVideoTrackDetector(),
+    new FrozenVideoTrackDetector(),
   ],
   getStatsInterval: 10_000, // set custom stats parsing interval
   onIssues: (payload: IssueDetectorResult) => {

src/WebRTCIssueDetector.ts

@@ -23,7 +23,7 @@ import {
   OutboundNetworkIssueDetector,
   QualityLimitationsIssueDetector,
   UnknownVideoDecoderImplementationDetector,
-  DeadVideoTrackDetector,
+  FrozenVideoTrackDetector,
 } from './detectors';
 import { CompositeRTCStatsParser, RTCStatsParser } from './parser';
 import createLogger from './utils/logger';
@@ -66,7 +66,7 @@ class WebRTCIssueDetector {
       new NetworkMediaSyncIssueDetector(),
       new AvailableOutgoingBitrateIssueDetector(),
       new UnknownVideoDecoderImplementationDetector(),
-      new DeadVideoTrackDetector(),
+      new FrozenVideoTrackDetector(),
     ];
 
     this.networkScoresCalculator = params.networkScoresCalculator ?? new DefaultNetworkScoresCalculator();

src/detectors/DeadVideoTrackDetector.ts renamed to src/detectors/FrozenVideoTrackDetector.ts

@@ -7,19 +7,19 @@ import {
 } from '../types';
 import BaseIssueDetector from './BaseIssueDetector';
 
-interface DeadVideoTrackDetectorParams {
+interface FrozenVideoTrackDetectorParams {
   timeoutMs?: number;
   framesDroppedThreshold?: number;
 }
 
-class DeadVideoTrackDetector extends BaseIssueDetector {
+class FrozenVideoTrackDetector extends BaseIssueDetector {
   readonly #lastMarkedAt = new Map<string, number>();
 
   readonly #timeoutMs: number;
 
   readonly #framesDroppedThreshold: number;
 
-  constructor(params: DeadVideoTrackDetectorParams = {}) {
+  constructor(params: FrozenVideoTrackDetectorParams = {}) {
     super();
     this.#timeoutMs = params.timeoutMs ?? 10_000;
     this.#framesDroppedThreshold = params.framesDroppedThreshold ?? 0.5;
@@ -69,6 +69,7 @@ class DeadVideoTrackDetector extends BaseIssueDetector {
         return;
       }
 
+      // We skip it when ratio is too low because it should be handled by FramesDroppedIssueDetector
       if (ratioFramesDropped >= this.#framesDroppedThreshold) {
         return;
       }
@@ -132,4 +133,4 @@ class DeadVideoTrackDetector extends BaseIssueDetector {
   }
 }
 
-export default DeadVideoTrackDetector;
+export default FrozenVideoTrackDetector;

src/detectors/index.ts

@@ -7,4 +7,4 @@ export { default as NetworkMediaSyncIssueDetector } from './NetworkMediaSyncIssu
 export { default as OutboundNetworkIssueDetector } from './OutboundNetworkIssueDetector';
 export { default as QualityLimitationsIssueDetector } from './QualityLimitationsIssueDetector';
 export { default as UnknownVideoDecoderImplementationDetector } from './UnknownVideoDecoderImplementationDetector';
-export { default as DeadVideoTrackDetector } from './DeadVideoTrackDetector';
+export { default as FrozenVideoTrackDetector } from './FrozenVideoTrackDetector';