From 209934d4c3878bbc14592cbd4b8c3b25ec52a659 Mon Sep 17 00:00:00 2001
From: brandonocasey <brandonocasey@gmail.com>
Date: Tue, 7 Jan 2020 16:34:44 -0500
Subject: [PATCH 1/6] chore: update netlify/stats to show bug

---
 index.html             |  1 +
 scripts/netlify.js     | 16 +++++++++-------
 utils/stats/index.html | 18 ++++++++++--------
 3 files changed, 20 insertions(+), 15 deletions(-)

diff --git a/index.html b/index.html
index 2f2b3bb2f..e2832a90b 100644
--- a/index.html
+++ b/index.html
@@ -51,6 +51,7 @@
     <li><a href="test/">Run unit tests in browser.</a></li>
     <li><a href="docs/api/">Read generated docs.</a></li>
     <li><a href="examples">Browse Examples</a></li>
+    <li><a href="utils/stats/">Stats</a></li>
   </ul>
 
   <script src="node_modules/video.js/dist/alt/video.core.js"></script>
diff --git a/scripts/netlify.js b/scripts/netlify.js
index 86f9fc996..3bd36d8dd 100644
--- a/scripts/netlify.js
+++ b/scripts/netlify.js
@@ -1,12 +1,14 @@
 const path = require('path');
 const sh = require('shelljs');
-const fs = require('fs');
-
-const vjs = 'node_modules/video.js/dist/alt/video.core.js';
-const vjsCss = 'node_modules/video.js/dist/video-js.css';
-const eme = 'node_modules/videojs-contrib-eme/dist/videojs-contrib-eme.js';
 const deployDir = 'deploy';
-const files = [vjs, vjsCss, eme];
+
+const files = [
+  'node_modules/video.js/dist/video-js.css',
+  'node_modules/video.js/dist/alt/video.core.js',
+  'node_modules/videojs-contrib-eme/dist/videojs-contrib-eme.js',
+  'node_modules/videojs-contrib-quality-levels/dist/videojs-contrib-quality-levels.js',
+  'node_modules/d3/d3.min.js'
+];
 
 // cleanup previous deploy
 sh.rm('-rf', deployDir);
@@ -20,5 +22,5 @@ files
 
 // copy over files, dist, and html files
 files
-.concat('dist', 'index.html', 'index.min.html')
+.concat('dist', 'index.html', 'index.min.html', 'utils')
 .forEach((file) => sh.cp('-r', file, path.join(deployDir, file)));
diff --git a/utils/stats/index.html b/utils/stats/index.html
index ea3173757..6597ae974 100644
--- a/utils/stats/index.html
+++ b/utils/stats/index.html
@@ -20,7 +20,7 @@
 
   <!-- player stats visualization -->
   <link href="stats.css" rel="stylesheet">
-  <script src="/node_modules/d3/d3.min.js"></script>
+  <script src="../../node_modules/d3/d3.min.js"></script>
 
   <style>
     body {
@@ -347,12 +347,7 @@ <h3>Timed Metadata</h3>
 
         player.ready(function() {
           var qualityLevels = player.qualityLevels();
-
-          qualityLevels.on('addqualitylevel', function(event) {
-            createQualityButton(event.qualityLevel, qualityButtons);
-          });
-
-          qualityLevels.on('change', function(event) {
+          var setupLevels = function() {
             for (var i = 0; i < qualityLevels.length; i++) {
               var level = qualityLevels[i];
               var button = document.getElementById('quality-level-' + level.id);
@@ -360,10 +355,17 @@ <h3>Timed Metadata</h3>
               button.classList.remove('selected');
             }
 
-            var selected = qualityLevels[event.selectedIndex];
+            var selected = qualityLevels[qualityLevels.selectedIndex];
             var button = document.getElementById('quality-level-' + selected.id);
             button.classList.add('selected');
+          };
+
+          qualityLevels.on('addqualitylevel', function(event) {
+            createQualityButton(event.qualityLevel, qualityButtons);
           });
+
+          qualityLevels.on('change', setupLevels);
+          setupLevels();
         });
       });
     }

From a84cc14c1e2f8554fecb123d3ea793e80c26188e Mon Sep 17 00:00:00 2001
From: brandonocasey <brandonocasey@gmail.com>
Date: Tue, 7 Jan 2020 16:51:44 -0500
Subject: [PATCH 2/6] stats page race condition fix

---
 utils/stats/index.html | 32 ++++++++++++++------------------
 1 file changed, 14 insertions(+), 18 deletions(-)

diff --git a/utils/stats/index.html b/utils/stats/index.html
index 6597ae974..3537aa84b 100644
--- a/utils/stats/index.html
+++ b/utils/stats/index.html
@@ -345,27 +345,23 @@ <h3>Timed Metadata</h3>
         videojs.Hls.displayStats(document.querySelector('.switching-stats'), player);
         videojs.Hls.displayCues(document.querySelector('.segment-timeline'), player);
 
-        player.ready(function() {
-          var qualityLevels = player.qualityLevels();
-          var setupLevels = function() {
-            for (var i = 0; i < qualityLevels.length; i++) {
-              var level = qualityLevels[i];
-              var button = document.getElementById('quality-level-' + level.id);
-
-              button.classList.remove('selected');
-            }
+        var qualityLevels = player.qualityLevels();
 
-            var selected = qualityLevels[qualityLevels.selectedIndex];
-            var button = document.getElementById('quality-level-' + selected.id);
-            button.classList.add('selected');
-          };
+        qualityLevels.on('addqualitylevel', function(event) {
+          createQualityButton(event.qualityLevel, qualityButtons);
+        });
 
-          qualityLevels.on('addqualitylevel', function(event) {
-            createQualityButton(event.qualityLevel, qualityButtons);
-          });
+        qualityLevels.on('change', function(event) {
+          for (var i = 0; i < qualityLevels.length; i++) {
+            var level = qualityLevels[i];
+            var button = document.getElementById('quality-level-' + level.id);
+
+            button.classList.remove('selected');
+          }
 
-          qualityLevels.on('change', setupLevels);
-          setupLevels();
+          var selected = qualityLevels[event.selectedIndex];
+          var button = document.getElementById('quality-level-' + selected.id);
+          button.classList.add('selected');
         });
       });
     }

From 89171abb6c5ac5e99c27d7016004a1bbd1e03460 Mon Sep 17 00:00:00 2001
From: brandonocasey <brandonocasey@gmail.com>
Date: Tue, 7 Jan 2020 16:56:20 -0500
Subject: [PATCH 3/6] fix: race condition preventing qualityLevels from being
 populated

---
 src/videojs-http-streaming.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/videojs-http-streaming.js b/src/videojs-http-streaming.js
index 8ac5ba66d..01eec6f79 100644
--- a/src/videojs-http-streaming.js
+++ b/src/videojs-http-streaming.js
@@ -716,7 +716,7 @@ class HlsHandler extends Component {
       this.tech_.trigger('progress');
     });
 
-    this.tech_.ready(() => this.setupQualityLevels_());
+    this.setupQualityLevels_();
 
     // do nothing if the tech has been disposed already
     // this can occur if someone sets the src in player.ready(), for instance

From fab164dd8810e5a8e194c19e2fabcc7060ddf4fa Mon Sep 17 00:00:00 2001
From: brandonocasey <brandonocasey@gmail.com>
Date: Wed, 8 Jan 2020 11:17:28 -0500
Subject: [PATCH 4/6] test fixes

---
 src/rendition-mixin.js        |  3 +++
 src/videojs-http-streaming.js | 22 +++++++++++++---------
 2 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/src/rendition-mixin.js b/src/rendition-mixin.js
index e0b406584..487b0cae6 100644
--- a/src/rendition-mixin.js
+++ b/src/rendition-mixin.js
@@ -92,6 +92,9 @@ let renditionSelectionMixin = function(hlsHandler) {
 
   // Add a single API-specific function to the HlsHandler instance
   hlsHandler.representations = () => {
+    if (!playlists || !playlists.master || !playlists.master.playlists) {
+      return [];
+    }
     return playlists
       .master
       .playlists
diff --git a/src/videojs-http-streaming.js b/src/videojs-http-streaming.js
index 01eec6f79..c717a48fd 100644
--- a/src/videojs-http-streaming.js
+++ b/src/videojs-http-streaming.js
@@ -737,17 +737,21 @@ class HlsHandler extends Component {
   setupQualityLevels_() {
     let player = videojs.players[this.tech_.options_.playerId];
 
-    if (player && player.qualityLevels) {
-      this.qualityLevels_ = player.qualityLevels();
+    // if there isn't a player or there isn't a qualityLevels plugin
+    // or qualityLevels_ listeners have already been setup, do nothing.
+    if (!player || !player.qualityLevels || this.qualityLevels_) {
+      return;
+    }
 
-      this.masterPlaylistController_.on('selectedinitialmedia', () => {
-        handleHlsLoadedMetadata(this.qualityLevels_, this);
-      });
+    this.qualityLevels_ = player.qualityLevels();
 
-      this.playlists.on('mediachange', () => {
-        handleHlsMediaChange(this.qualityLevels_, this.playlists);
-      });
-    }
+    this.masterPlaylistController_.on('selectedinitialmedia', () => {
+      handleHlsLoadedMetadata(this.qualityLevels_, this);
+    });
+
+    this.playlists.on('mediachange', () => {
+      handleHlsMediaChange(this.qualityLevels_, this.playlists);
+    });
   }
 
   /**

From 7abf951544081abbbf3251b0411d81c72bf938f6 Mon Sep 17 00:00:00 2001
From: brandonocasey <brandonocasey@gmail.com>
Date: Wed, 8 Jan 2020 12:08:05 -0500
Subject: [PATCH 5/6] unskip test

---
 test/videojs-http-streaming.test.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/test/videojs-http-streaming.test.js b/test/videojs-http-streaming.test.js
index 9255b0227..e9980a7d9 100644
--- a/test/videojs-http-streaming.test.js
+++ b/test/videojs-http-streaming.test.js
@@ -3094,8 +3094,7 @@ QUnit.test('passes useCueTags hls option to master playlist controller', functio
   videojs.options.hls = origHlsOptions;
 });
 
-// TODO: This test fails intermittently. Turn on when fixed to always pass.
-QUnit.skip('populates quality levels list when available', function(assert) {
+QUnit.test('populates quality levels list when available', function(assert) {
   this.player.src({
     src: 'manifest/master.m3u8',
     type: 'application/vnd.apple.mpegurl'

From 99c5fd29894eb0c97edeb65f46b4dc5d9b1c1b20 Mon Sep 17 00:00:00 2001
From: brandonocasey <brandonocasey@gmail.com>
Date: Wed, 8 Jan 2020 14:11:20 -0500
Subject: [PATCH 6/6] cr

---
 scripts/netlify.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/netlify.js b/scripts/netlify.js
index 3bd36d8dd..2cbbc0078 100644
--- a/scripts/netlify.js
+++ b/scripts/netlify.js
@@ -20,7 +20,7 @@ files
 .map((file) => path.dirname(file))
 .forEach((dir) => sh.mkdir('-p', path.join(deployDir, dir)));
 
-// copy over files, dist, and html files
+// copy files/folders to deploy dir
 files
 .concat('dist', 'index.html', 'index.min.html', 'utils')
 .forEach((file) => sh.cp('-r', file, path.join(deployDir, file)));