fix: switch off in-manifest caption support (#8)

videojs · Nov 29, 2017 · 15712c6 · 15712c6
1 parent 973dba0
commit 15712c6
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/src/toM3u8.js b/src/toM3u8.js
@@ -141,7 +141,8 @@ export const toM3u8 = dashPlaylists => {
     master.mediaGroups.AUDIO.audio = organizeAudioPlaylists(audioPlaylists);
   }
 
-  if (vttPlaylists.length) {
+  // TODO: vtt playlists not yet supported
+  if (vttPlaylists.length && false) {
     master.mediaGroups.SUBTITLES.subs = organizeVttPlaylists(vttPlaylists);
   }
 

diff --git a/test/manifests/maat_vtt_segmentTemplate.js b/test/manifests/maat_vtt_segmentTemplate.js
@@ -101,6 +101,9 @@ export const parsedManifest = {
       }
     },
     ['CLOSED-CAPTIONS']: {},
+    // Subtitles feature is turned off
+    SUBTITLES: {},
+    /*
     SUBTITLES: {
       subs: {
         en: {
@@ -141,6 +144,7 @@ export const parsedManifest = {
         }
       }
     },
+    */
     VIDEO: {}
   },
   playlists: [{

diff --git a/test/toM3u8.test.js b/test/toM3u8.test.js
@@ -88,6 +88,9 @@ QUnit.test('playlists', function(assert) {
         }
       },
       ['CLOSED-CAPTIONS']: {},
+      // Subtitles feature is turned off
+      SUBTITLES: {},
+      /*
       SUBTITLES: {
         subs: {
           text: {
@@ -110,6 +113,7 @@ QUnit.test('playlists', function(assert) {
           }
         }
       },
+      */
       VIDEO: {}
     },
     playlists: [{