From fd19b3faa1d9bb26123498bb4d984be1491d7418 Mon Sep 17 00:00:00 2001
From: mister-ben <1676039+mister-ben@users.noreply.github.com>
Date: Wed, 30 Oct 2024 21:01:21 +0100
Subject: [PATCH] fix: Ensure native segment metadata track has mode hidden

---
 index.html                       |  5 +++++
 scripts/index.js                 |  7 +++++--
 src/playlist-controller.js       |  2 ++
 test/playlist-controller.test.js | 20 ++++++++++++++++++++
 4 files changed, 32 insertions(+), 2 deletions(-)

diff --git a/index.html b/index.html
index d2940c3d9..ee6843d5b 100644
--- a/index.html
+++ b/index.html
@@ -192,6 +192,11 @@
           <label class="form-check-label" for="forced-subtitles">Use Forced Subtitles (reloads player)</label>
         </div>
 
+        <div class="form-check">
+          <input id="native-text-tracks" type="checkbox" class="form-check-input">
+          <label class="form-check-label" for="native-text-tracks">Use native text tracks (reloads player)</label>
+        </div>
+
         <div class="input-group">
           <span class="input-group-text"><label for=preload>Preload (reloads player)</label></span>
           <select id=preload class="form-select">
diff --git a/scripts/index.js b/scripts/index.js
index 4b66316d6..db568962a 100644
--- a/scripts/index.js
+++ b/scripts/index.js
@@ -474,7 +474,8 @@
     'use-mms',
     'preload',
     'mirror-source',
-    'forced-subtitles'
+    'forced-subtitles',
+    'native-text-tracks'
   ].forEach(function(name) {
     stateEls[name] = document.getElementById(name);
   });
@@ -528,7 +529,8 @@
       'network-info',
       'dts-offset',
       'exact-manifest-timings',
-      'forced-subtitles'
+      'forced-subtitles',
+      'native-text-tracks'
     ].forEach(function(name) {
       stateEls[name].addEventListener('change', function(event) {
         saveState();
@@ -603,6 +605,7 @@
           liveui: stateEls.liveui.checked,
           enableSourceset: mirrorSource,
           html5: {
+            nativeTextTracks: getInputValue(stateEls['native-text-tracks']),
             vhs: {
               overrideNative: getInputValue(stateEls['override-native']),
               experimentalUseMMS: getInputValue(stateEls['use-mms']),
diff --git a/src/playlist-controller.js b/src/playlist-controller.js
index 6f4f97130..7f4f6ff7a 100644
--- a/src/playlist-controller.js
+++ b/src/playlist-controller.js
@@ -246,6 +246,8 @@ export class PlaylistController extends videojs.EventTarget {
       label: 'segment-metadata'
     }, false).track;
 
+    this.segmentMetadataTrack_.mode = 'hidden';
+
     this.decrypter_ = new Decrypter();
     this.sourceUpdater_ = new SourceUpdater(this.mediaSource);
     this.inbandTextTracks_ = {};
diff --git a/test/playlist-controller.test.js b/test/playlist-controller.test.js
index 0f555d556..51539473d 100644
--- a/test/playlist-controller.test.js
+++ b/test/playlist-controller.test.js
@@ -7729,3 +7729,23 @@ QUnit.test('ManagedMediaSource startstreaming and endstreaming events start and
 
   mms.restore();
 });
+
+QUnit.module('Native tracks', {
+  beforeEach(assert) {
+    this.playerOptions = {
+      html5: {
+        nativeTextTracks: true
+      }
+    };
+    sharedHooks.beforeEach.call(this, assert);
+    this.pc = this.playlistController;
+
+  },
+  afterEach(assert) {
+    sharedHooks.afterEach.call(this, assert);
+  }
+});
+
+QUnit.test('Native text track mode is hidden', function(assert) {
+  assert.strictEqual(this.pc.segmentMetadataTrack_.mode, 'hidden', 'track mode is hidden');
+});