From 58604795f8f96cc6fed1103835559c5386fec0ae Mon Sep 17 00:00:00 2001 From: Richard Brandon Date: Mon, 18 Jul 2016 15:35:51 -0400 Subject: [PATCH] @rbran100 checked src and currentSrc in handleTechReady to work around mixed content issues in chrome. closes #3287 --- CHANGELOG.md | 1 + src/js/player.js | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 03d7c6090..95c2debdf 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -24,6 +24,7 @@ CHANGELOG * @mister-ben updated language to inherit correctly and respect the attribute on the player ([view](https://github.com/videojs/video.js/pull/3426)) * @sashyro fixed nativeControlsForTouch option ([view](https://github.com/videojs/video.js/pull/3410)) * @tbasse fixed techCall null check against tech ([view](https://github.com/videojs/video.js/pull/2676)) +* @rbran100 checked src and currentSrc in handleTechReady to work around mixed content issues in chrome ([view](https://github.com/videojs/video.js/pull/3287)) -------------------- diff --git a/src/js/player.js b/src/js/player.js index 518b95156..ec5c6877f 100644 --- a/src/js/player.js +++ b/src/js/player.js @@ -792,7 +792,7 @@ class Player extends Component { // In Safari (5.1.1), when we move the video element into the container div, autoplay doesn't work. // In Chrome (15), if you have autoplay + a poster + no controls, the video gets hidden (but audio plays) // This fixes both issues. Need to wait for API, so it updates displays correctly - if (this.src() && this.tag && this.options_.autoplay && this.paused()) { + if ((this.src() || this.currentSrc()) && this.tag && this.options_.autoplay && this.paused()) { try { delete this.tag.poster; // Chrome Fix. Fixed in Chrome v16. }