videomail-client 3.0.1 → 5.0.0

package/src/js/util/browser.js

@@ -10,7 +10,6 @@ const Browser = function (options) {
   const edgeDownload = 'https://www.microsoft.com/en-us/download/details.aspx?id=48126'
   const chromeDownload = 'http://www.google.com/chrome/'
   const chromiumDownload = 'http://www.chromium.org/getting-involved/download-chromium'
-  const browseHappyLink = 'http://browsehappy.com'
   const ua = defined(
     options.fakeUaString,
     typeof window !== 'undefined' && window.navigator && window.navigator.userAgent,
@@ -148,35 +147,6 @@ const Browser = function (options) {
     return warning
   }
 
-  function getPlaybackWarning() {
-    let warning = getRecommendation()
-
-    if (!warning) {
-      warning =
-        '<a href="' +
-        browseHappyLink +
-        '" target="_blank">Upgrading your browser</a> might help.'
-    }
-
-    return warning
-  }
-
-  function canPlayType(video, type) {
-    let canPlayType
-
-    if (video && video.canPlayType) {
-      canPlayType = video.canPlayType('video/' + type)
-    }
-
-    // definitely cannot be played here
-    if (canPlayType === '') {
-      return false
-    }
-
-    return canPlayType
-  }
-
-  // just temporary
   this.canRecord = function () {
     const hasNavigator = typeof navigator !== 'undefined'
     let canRecord = false
@@ -257,28 +227,6 @@ const Browser = function (options) {
     return err
   }
 
-  this.checkPlaybackCapabilities = function (video) {
-    options.debug('Browser: checkPlaybackCapabilities()')
-
-    let err
-    let message
-
-    if (!video) {
-      message = 'No HTML5 support for video tag!'
-    } else if (!this.getVideoType(video)) {
-      message = 'Your old browser cannot support modern video codecs'
-    } else if (!video.setAttribute) {
-      // fixes "Not implemented" error on older browsers
-      message = 'Unable to set video attributes in your old browser'
-    }
-
-    if (message) {
-      err = VideomailError.create(message, getPlaybackWarning(), options)
-    }
-
-    return err
-  }
-
   this.checkBufferTypes = function () {
     let err
 
@@ -293,6 +241,21 @@ const Browser = function (options) {
     return err
   }
 
+  function canPlayType(video, type) {
+    let canPlayType
+
+    if (video && video.canPlayType) {
+      canPlayType = video.canPlayType('video/' + type)
+    }
+
+    // definitely cannot be played here
+    if (canPlayType === '') {
+      return false
+    }
+
+    return canPlayType
+  }
+
   this.getVideoType = function (video) {
     if (!video) {
       // no type without video
@@ -300,18 +263,16 @@ const Browser = function (options) {
     }
 
     if (!videoType) {
-      // there is a bug in canPlayType within chrome for mp4
-      if (canPlayType(video, 'mp4') && !chromeBased) {
-        videoType = 'mp4'
-      } else if (canPlayType(video, 'webm')) {
+      if (canPlayType(video, 'webm')) {
         videoType = 'webm'
+      } else if (canPlayType(video, 'mp4')) {
+        videoType = 'mp4'
       }
     }
 
     if (!videoType || videoType === '') {
-      // pick this one as a fallback since it's widely used and has
-      // greatly improved
-      videoType = 'mp4'
+      // just as a fallback
+      videoType = 'webm'
     }
 
     return videoType

package/src/js/wrappers/visuals/recorder.js

@@ -237,6 +237,8 @@ const Recorder = function (visuals, replay, defaultOptions = {}) {
 
     key = args.key
 
+    // We are not serving MP4 videos anymore due to licensing but are keeping code
+    // for compatibility and documentation
     if (args.mp4) {
       replay.setMp4Source(
         args.mp4 + Constants.SITE_NAME_LABEL + '/' + options.siteName + '/videomail.mp4',

package/src/js/wrappers/visuals/replay.js

@@ -1,14 +1,13 @@
-import util from 'util'
-import h from 'hyperscript'
-import hidden from 'hidden'
 import addEventListenerWithOptions from 'add-eventlistener-with-options'
-
-import Events from './../../events'
-import Browser from './../../util/browser'
-import EventEmitter from './../../util/eventEmitter'
-import VideomailError from './../../util/videomailError'
-
+import hidden from 'hidden'
+import h from 'hyperscript'
 import enableInlineVideo from 'iphone-inline-video'
+import util from 'util'
+
+import Events from '../../events'
+import Browser from '../../util/browser'
+import EventEmitter from '../../util/eventEmitter'
+import VideomailError from '../../util/videomailError'
 
 const Replay = function (parentElement, options) {
   EventEmitter.call(this, options, 'Replay')