flac support: don't duplicate regex needlessly

Author: jprjr

src/controller/buffer-controller.ts

@@ -31,7 +31,6 @@ import type { LevelDetails } from '../loader/level-details';
 
 const MediaSource = getMediaSource();
 const VIDEO_CODEC_PROFILE_REPACE = /([ha]vc.)(?:\.[^.,]+)+/;
-const AUDIO_CODEC_REGEXP = /flac|opus/gi;
 
 export default class BufferController implements ComponentAPI {
   // The level details used to determine duration, target-duration and live
@@ -265,11 +264,8 @@ export default class BufferController implements ComponentAPI {
           );
           if (currentCodec !== nextCodec) {
             let trackCodec = levelCodec || codec;
-            if (trackName.indexOf('audio') !== -1) {
-              trackCodec = trackCodec.replace(
-                AUDIO_CODEC_REGEXP,
-                getCodecCompatibleName
-              );
+            if (trackName.slice(0, 5) === 'audio') {
+              trackCodec = getCodecCompatibleName(trackCodec);
             }
             const mimeType = `${container};codecs=${trackCodec}`;
             this.appendChangeType(trackName, mimeType);
@@ -758,8 +754,8 @@ export default class BufferController implements ComponentAPI {
         // use levelCodec as first priority
         let codec = track.levelCodec || track.codec;
         if (codec) {
-          if (trackName.indexOf('audio') !== -1) {
-            codec = codec.replace(AUDIO_CODEC_REGEXP, getCodecCompatibleName);
+          if (trackName.slice(0, 5) === 'audio') {
+            codec = getCodecCompatibleName(codec);
           }
         }
         const mimeType = `${track.container};codecs=${codec}`;

src/controller/level-controller.ts

@@ -26,8 +26,6 @@ import ContentSteeringController from './content-steering-controller';
 
 let chromeOrFirefox: boolean;
 
-const AUDIO_CODEC_REGEXP = /flac|opus/gi;
-
 export default class LevelController extends BasePlaylistController {
   private _levels: Level[] = [];
   private _firstLevel: number = -1;
@@ -125,10 +123,7 @@ export default class LevelController extends BasePlaylistController {
       }
 
       if (levelParsed.audioCodec) {
-        levelParsed.audioCodec = levelParsed.audioCodec.replace(
-          AUDIO_CODEC_REGEXP,
-          getCodecCompatibleName
-        );
+        levelParsed.audioCodec = getCodecCompatibleName(levelParsed.audioCodec);
       }
 
       const {

src/utils/codecs.ts

@@ -118,6 +118,9 @@ function getCodecCompatibleNameLower(
   return lowerCaseCodec;
 }
 
+const AUDIO_CODEC_REGEXP = /flac|opus/i;
 export function getCodecCompatibleName(codec: string): string {
-  return getCodecCompatibleNameLower(codec.toLowerCase() as LowerCaseCodecType);
+  return codec.replace(AUDIO_CODEC_REGEXP, (m) =>
+    getCodecCompatibleNameLower(m.toLowerCase() as LowerCaseCodecType)
+  );
 }