Fix for Dash-Industry-Forum#1437

dsparacio · Jun 7, 2016 · dfbc386 · dfbc386
1 parent a9d3bfc
commit dfbc386
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 10 deletions.
diff --git a/src/streaming/controllers/AbrController.js b/src/streaming/controllers/AbrController.js
@@ -46,7 +46,7 @@ const ABANDON_LOAD = 'abandonload';
 const ALLOW_LOAD = 'allowload';
 const DEFAULT_VIDEO_BITRATE = 1000;
 const DEFAULT_AUDIO_BITRATE = 100;
-const QUALITY_DEFAULT = -1;
+const QUALITY_DEFAULT = 0;
 
 function AbrController() {
 

diff --git a/src/streaming/controllers/BufferController.js b/src/streaming/controllers/BufferController.js
@@ -92,14 +92,14 @@ function BufferController(config) {
         clearBufferTimeout;
 
     function setup() {
-        requiredQuality = -1;
-        currentQuality = -1;
+        requiredQuality = AbrController.QUALITY_DEFAULT;
+        currentQuality = AbrController.QUALITY_DEFAULT;
         isBufferingCompleted = false;
         bufferLevel = 0;
         bufferTarget = 0;
         criticalBufferLevel = Number.POSITIVE_INFINITY;
-        maxAppendedIndex = -1;
-        lastIndex = -1;
+        maxAppendedIndex = 0;
+        lastIndex = 0;
         buffer = null;
         bufferState = BUFFER_EMPTY;
         wallclockTicked = 0;
@@ -713,10 +713,10 @@ function BufferController(config) {
 
         criticalBufferLevel = Number.POSITIVE_INFINITY;
         bufferState = BUFFER_EMPTY;
-        currentQuality = -1;
-        lastIndex = -1;
-        maxAppendedIndex = -1;
-        requiredQuality = 0;
+        currentQuality = AbrController.QUALITY_DEFAULT;
+        requiredQuality = AbrController.QUALITY_DEFAULT;
+        lastIndex = 0;
+        maxAppendedIndex = 0;
         appendedBytesInfo = null;
         appendingMediaChunk = false;
         isBufferingCompleted = false;

diff --git a/src/streaming/controllers/ScheduleController.js b/src/streaming/controllers/ScheduleController.js
@@ -86,12 +86,14 @@ function ScheduleController(config) {
         bufferLevelRule,
         nextFragmentRequestRule,
         scheduleWhilePaused,
+        initialQualityChange,
         qualityChangeInProgress,
         renderTimeCheckInterval;
 
     function setup() {
         qualityChangeInProgress = false;
         initialPlayback = true;
+        initialQualityChange = true;
         isStopped = false;
         playListMetrics = null;
         playListTraceMetrics = null;
@@ -275,9 +277,11 @@ function ScheduleController(config) {
             throw new Error('Unexpected error! - currentRepresentationInfo is null or undefined');
         }
 
-        if (e.oldQuality !== AbrController.QUALITY_DEFAULT) { //blocks init quality change
+        if (!initialQualityChange) { //blocks init quality change
             qualityChangeInProgress = true;
             eventBus.trigger(Events.QUALITY_CHANGE_START, {mediaType: e.mediaType, newQuality: e.newQuality,  oldQuality: e.oldQuality});
+        } else {
+            initialQualityChange = false;
         }
 
         clearPlayListTraceMetrics(new Date(), PlayListTrace.REPRESENTATION_SWITCH_STOP_REASON);