Properly close the video file if rendering is aborted

cleanflight · Jan 12, 2017 · d51d402 · d51d402
1 parent 99da72c
commit d51d402
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/app/js/flightlog_video_renderer.js b/app/js/flightlog_video_renderer.js
@@ -133,7 +133,7 @@ function FlightLogVideoRenderer(flightLog, logParameters, videoOptions) {
         that.emit("complete", success, frameCount);
     }
 
-    function finishRender() {
+    function finishRender(success) {
         var
             complete;
 
@@ -146,7 +146,7 @@ function FlightLogVideoRenderer(flightLog, logParameters, videoOptions) {
         }
 
         complete.then(function() {
-            notifyCompletion(true, frameIndex);
+            notifyCompletion(success, frameIndex);
         });
     }
 
@@ -162,7 +162,7 @@ function FlightLogVideoRenderer(flightLog, logParameters, videoOptions) {
             framesToRender = Math.min(workChunkSize, frameCount - frameIndex);
 
         if (cancel) {
-            notifyCompletion(false);
+            finishRender(false);
             return;
         }
 
@@ -171,7 +171,7 @@ function FlightLogVideoRenderer(flightLog, logParameters, videoOptions) {
                 that.emit("progress", frameIndex, frameCount, that.getWrittenSize());
 
                 if (frameIndex >= frameCount) {
-                    finishRender();
+                    finishRender(true);
                 } else {
                     setTimeout(renderChunk, 0);
                 }