Merge pull request #254 from jfboeve/master

Addition of getter and Handling promise rejection

Author: michielvandergeest

package-lock.json

@@ -23,6 +23,7 @@
   "dependencies": {
     "@babel/polyfill": "^7.11.5",
     "@lightningjs/core": "*",
+    "@michieljs/execute-as-promise": "^1.0.0",
     "deepmerge": "^4.2.2",
     "localCookie": "github:WebPlatformForEmbedded/localCookie",
     "shelljs": "^0.8.4",

package.json

@@ -17,6 +17,8 @@
  * limitations under the License.
  */
 
+import executeAsPromise from '@michieljs/execute-as-promise'
+
 import Metrics from '../Metrics'
 import Log from '../Log'
 import Ads from '../Ads'
@@ -233,11 +235,15 @@ const videoPlayerPlugin = {
         state.playingAds = true
         ads.prerolls().then(() => {
           state.playingAds = false
-          loader(url, videoEl, config).then(() => {
-            registerEventListeners()
-            this.show()
-            this.play()
-          })
+          loader(url, videoEl, config)
+            .then(() => {
+              registerEventListeners()
+              this.show()
+              this.play()
+            })
+            .catch(e => {
+              fireOnConsumer('error', { videoElement: videoEl, event: e })
+            })
         })
       })
     }
@@ -279,7 +285,9 @@ const videoPlayerPlugin = {
   play() {
     if (!this.canInteract) return
     if (textureMode === true) videoTexture.start()
-    videoEl.play()
+    executeAsPromise(videoEl.play, null, videoEl).catch(e => {
+      fireOnConsumer('error', { videoElement: videoEl, event: e })
+    })
   },
 
   pause() {
@@ -424,6 +432,10 @@ const videoPlayerPlugin = {
   get _videoEl() {
     return videoEl
   },
+
+  get _consumer() {
+    return consumer
+  },
 }
 
 export default autoSetupMixin(videoPlayerPlugin, () => {