vmsg plugin: add destroy (#331)

* vmsg plugin: add destroy

* add changelog entry

CHANGES.md

@@ -1,6 +1,12 @@
 videojs-record changelog
 ========================
 
+3.4.2 - unreleased
+------------------
+
+- vmsg plugin: add `destroy`
+
+
 3.4.1 - 2019/02/09
 ------------------
 
@@ -15,8 +21,8 @@ videojs-record changelog
 - Support video recording in Safari (Technology Preview 73 or newer) (#181)
 - Bump required version for recordrtc to 5.5.3 for Safari video recording
   support (#320)
-- Added example for changing video source (#223)
-- Added ability to permanently hide custom UI elements (e.g. `recordToggle`) 
+- Add example for changing video source (#223)
+- Add ability to permanently hide custom UI elements (e.g. `recordToggle`) 
 
 
 3.3.0 - 2019/01/10

src/js/engine/record-engine.js

@@ -57,6 +57,11 @@ class RecordEngine extends Component {
         }
     }
 
+    /**
+     * Destroy engine.
+     */
+    destroy() {}
+
     /**
      * Add filename and timestamp to recorded file object.
      *

src/js/engine/record-rtc.js

@@ -68,7 +68,14 @@ class RecordRTCEngine extends RecordEngine {
     dispose() {
         super.dispose();
 
-        if (typeof this.engine.destroy === 'function') {
+        this.destroy();
+    }
+
+    /**
+     * Destroy engine.
+     */
+    destroy() {
+        if (this.engine && typeof this.engine.destroy === 'function') {
             this.engine.destroy();
         }
     }
@@ -163,7 +170,6 @@ class RecordRTCEngine extends RecordEngine {
 
             // notify listeners
             this.trigger('recordComplete');
-
         });
     }
 }

src/js/plugins/vmsg-plugin.js

@@ -84,6 +84,15 @@ class VmsgEngine extends RecordEngine {
         }
     }
 
+    /**
+     * Destroy engine.
+     */
+    destroy() {
+        if (this.engine && typeof this.engine.close === 'function') {
+            this.engine.close();
+        }
+    }
+
     /**
      * Continuous encoding of audio data.
      * @private

src/js/videojs.record.js

@@ -1127,6 +1127,7 @@ class Record extends Plugin {
         // prevent callbacks if recording is in progress
         if (this.engine) {
             this.engine.dispose();
+            this.engine.destroy();
             this.engine.off('recordComplete', this.engineStopCallback);
         }