videomail-client 5.2.0 → 5.2.2

package/TODO.md

@@ -1,17 +1,17 @@
 ### TODOs
 | Filename | line # | TODO
 |:------|:------:|:------
+| src/js/wrappers/container.js | 334 | figure out how to fire dom's onload event again
+| src/js/wrappers/container.js | 335 | or how to run all the scripts over again
+| src/js/wrappers/optionsWrapper.js | 27 | fix this, it's not really an option
 | src/js/util/audioRecorder.js | 11 | code needs rewrite
 | src/js/util/eventEmitter.js | 6 | MAKE EVENT EMITTING IN DESPOT NOT GLOBAL BUT BY CONTAINER ID INSTEAD
 | src/js/util/eventEmitter.js | 43 | have this emitted through a configuration because it is pretty noisy
 | src/js/util/humanize.js | 4 | get rid of this class and use those imports directly
-| src/js/wrappers/container.js | 334 | figure out how to fire dom's onload event again
-| src/js/wrappers/container.js | 335 | or how to run all the scripts over again
-| src/js/wrappers/optionsWrapper.js | 27 | fix this, it's not really an option
-| src/js/wrappers/visuals/recorder.js | 584 | in https://github.com/binarykitchen/videomail-client/issues/142
-| src/js/wrappers/visuals/recorder.js | 621 | retry with navigator.getUserMedia_() maybe?
-| src/js/wrappers/visuals/recorder.js | 761 | commented out because for some reasons server does not accept such a long
-| src/js/wrappers/visuals/recorder.js | 766 | consider removing this later or have it for debug=1 only?
+| src/js/wrappers/visuals/recorder.js | 582 | in https://github.com/binarykitchen/videomail-client/issues/142
+| src/js/wrappers/visuals/recorder.js | 619 | retry with navigator.getUserMedia_() maybe?
+| src/js/wrappers/visuals/recorder.js | 759 | commented out because for some reasons server does
+| src/js/wrappers/visuals/recorder.js | 764 | consider removing this later or have it for debug=1 only?
 | src/js/wrappers/visuals/userMedia.js | 309 | consider removing that if it's not the case anymore (for better performance)
 | gulpfile.js | 1 | write this in ES6 once i have figured out how to
 | gulpfile.js | 58 | fix this, so that it also works when not minified, this

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "videomail-client",
-  "version": "5.2.0",
+  "version": "5.2.2",
   "description": "A wicked npm package to record videos directly in the browser, wohooo!",
   "author": "Michael Heuberger <michael.heuberger@binarykitchen.com>",
   "contributors": [

package/prototype/js/videomail-client.js

@@ -26004,7 +26004,7 @@ function wrappy (fn, cb) {
 },{}],345:[function(_dereq_,module,exports){
 module.exports={
   "name": "videomail-client",
-  "version": "5.2.0",
+  "version": "5.2.2",
   "description": "A wicked npm package to record videos directly in the browser, wohooo!",
   "author": "Michael Heuberger <michael.heuberger@binarykitchen.com>",
   "contributors": [
@@ -26375,6 +26375,8 @@ var _default = (0, _keymirror.default)({
   // webcam is recording
   STOPPING: null,
   // recording is being stopped (= preview)
+  STOPPED: null,
+  // recording has stopped
   PROGRESS: null,
   // start sending
   BEGIN_AUDIO_ENCODING: null,
@@ -30571,7 +30573,6 @@ var Recorder = function Recorder(visuals, replay) {
   var frame;
   var recordingBufferLength;
   var recordingBuffer;
-  var timeControlEnabled = Boolean(options.video.timeControl);
   function writeStream(buffer, opts) {
     if (stream) {
       if (stream.destroyed) {
@@ -31042,8 +31043,8 @@ var Recorder = function Recorder(visuals, replay) {
         args: args
       };
 
-      // todo commented out because for some reasons server does not accept such a long
-      // array of many log lines. to examine later.
+      // todo commented out because for some reasons server does
+      // not accept such a long array of many log lines. to examine later.
       //
       // add some useful debug info to examine weird stuff like this one
       // UnprocessableError: Unable to encode a video with FPS near zero.
@@ -31125,7 +31126,11 @@ var Recorder = function Recorder(visuals, replay) {
         recordingStats.samplesCount = samplesCount;
         recordingStats.sampleRate = userMedia.getAudioSampleRate();
       }
-      writeCommand('stop', recordingStats);
+      writeCommand('stop', recordingStats, function () {
+        self.emit(_events.default.STOPPED, {
+          recordingStats: recordingStats
+        });
+      });
 
       // beware, resetting will set framesCount to zero, so leave this here
       self.reset();
@@ -31221,7 +31226,6 @@ var Recorder = function Recorder(visuals, replay) {
       if (!self.isPaused() && stream && ctx) {
         if (framesCount === 0) {
           self.emit(_events.default.SENDING_FIRST_FRAME);
-          debug('Recorder: time control is', timeControlEnabled.toString());
         }
         framesCount++;
         ctx.drawImage(userMedia.getRawVisuals(), 0, 0, canvas.width, canvas.height);
@@ -31231,16 +31235,10 @@ var Recorder = function Recorder(visuals, replay) {
           throw _videomailError.default.create('Failed to extract webcam data.', options);
         }
         bytesSum += recordingBufferLength;
-        var frameBuffer;
-        if (timeControlEnabled) {
-          var timeControlBuffer = Buffer.from((0, _safeJsonStringify.default)({
-            frameNumber: framesCount,
-            milliseconds: Date.now()
-          }));
-          frameBuffer = Buffer.concat([recordingBuffer, timeControlBuffer]);
-        } else {
-          frameBuffer = recordingBuffer;
-        }
+        var frameControlBuffer = Buffer.from((0, _safeJsonStringify.default)({
+          frameNumber: framesCount
+        }));
+        var frameBuffer = Buffer.concat([recordingBuffer, frameControlBuffer]);
         writeStream(frameBuffer, {
           frameNumber: framesCount,
           onFlushedCallback: onFlushed