Fix tracks sorting in Offline tutorial sample code (#1609)

Sort bandwidths numerically. Fixes #1608
shaka-project · Oct 8, 2018 · 57e36ab · 57e36ab
1 parent 4b413ae
commit 57e36ab
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/AUTHORS b/AUTHORS
@@ -20,6 +20,7 @@ Bryan Huh <bhh1988@gmail.com>
 Esteban Dosztal <edosztal@gmail.com>
 Google Inc. <*@google.com>
 Edgeware AB <*@edgeware.tv>
+Gil Gonen <gil.gonen@gmail.com>
 Itay Kinnrot <Itay.Kinnrot@Kaltura.com>
 Jason Palmer <jason@jason-palmer.com>
 Jesper Haug Karsrud <jesper.karsrud@gmail.com>

diff --git a/CONTRIBUTORS b/CONTRIBUTORS
@@ -33,6 +33,7 @@ Donato Borrello <donato@jwplayer.com>
 Duc Pham <duc.pham@edgeware.tv>
 Esteban Dosztal <edosztal@gmail.com>
 François Beaufort <fbeaufort@google.com>
+Gil Gonen <gil.gonen@gmail.com>
 Itay Kinnrot <itay.kinnrot@kaltura.com>
 Jacob Trimble <modmaker@google.com>
 Jason Palmer <jason@jason-palmer.com>

diff --git a/docs/tutorials/offline.md b/docs/tutorials/offline.md
@@ -135,9 +135,9 @@ function selectTracks(tracks) {
   // Store the highest bandwidth variant.
   var found = tracks
       .filter(function(track) { return track.type == 'variant'; })
-      .sort(function(a, b) { return a.bandwidth > b.bandwidth; })
+      .sort(function(a, b) { return a.bandwidth - b.bandwidth; })
       .pop();
-  console.log('Offline Track: ' + found);
+  console.log('Offline Track bandwidth: ' + found.bandwidth);
   return [ found ];
 }
 
@@ -487,9 +487,9 @@ function selectTracks(tracks) {
   // Store the highest bandwidth variant.
   var found = tracks
       .filter(function(track) { return track.type == 'variant'; })
-      .sort(function(a, b) { return a.bandwidth > b.bandwidth; })
+      .sort(function(a, b) { return a.bandwidth - b.bandwidth; })
       .pop();
-  console.log('Offline Track: ' + found);
+  console.log('Offline Track bandwidth: ' + found.bandwidth);
   return [ found ];
 }