Move onremovetrack check into the Supports function.

videojs · Aug 12, 2015 · 887e45c · 887e45c
1 parent e90738c
commit 887e45c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/js/tech/html5.js b/src/js/tech/html5.js
@@ -893,6 +893,9 @@ Html5.supportsNativeTextTracks = function() {
   if (supportsTextTracks && browser.IS_FIREFOX) {
     supportsTextTracks = false;
   }
+  if (supportsTextTracks && ('onremovetrack' in Html5.TEST_VID.textTracks)) {
+    supportsTextTracks = false;
+  }
 
   return supportsTextTracks;
 };

diff --git a/test/unit/tracks/tracks.test.js b/test/unit/tracks/tracks.test.js
@@ -308,7 +308,7 @@ test('when switching techs, we should not get a new text track', function() {
   ok(htmltracks === flashtracks, 'the tracks are equal');
 });
 
-if (Html5.supportsNativeTextTracks() && ('onremovetrack' in Html5.TEST_VID.textTracks)) {
+if (Html5.supportsNativeTextTracks()) {
   test('listen to native remove and add track events in native text tracks', function(assert) {
     let done = assert.async();