diff options
author | Chocobozzz <me@florianbigard.com> | 2018-02-07 10:01:37 +0100 |
---|---|---|
committer | Chocobozzz <me@florianbigard.com> | 2018-02-07 10:03:11 +0100 |
commit | 481d35966fe49e9a5d3442b658e35403e9464322 (patch) | |
tree | 2c44537021dc515c04f5b5c4edf31d13b30f606b /client/src/assets/player/peertube-videojs-plugin.ts | |
parent | 0bc22f8d7ef0ddb4becae2e9453378d3a0f88218 (diff) | |
download | PeerTube-481d35966fe49e9a5d3442b658e35403e9464322.tar.gz PeerTube-481d35966fe49e9a5d3442b658e35403e9464322.tar.zst PeerTube-481d35966fe49e9a5d3442b658e35403e9464322.zip |
Fix video play on google chrome
Diffstat (limited to 'client/src/assets/player/peertube-videojs-plugin.ts')
-rw-r--r-- | client/src/assets/player/peertube-videojs-plugin.ts | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/client/src/assets/player/peertube-videojs-plugin.ts b/client/src/assets/player/peertube-videojs-plugin.ts index 5ccfdce36..125ef64a4 100644 --- a/client/src/assets/player/peertube-videojs-plugin.ts +++ b/client/src/assets/player/peertube-videojs-plugin.ts | |||
@@ -217,10 +217,15 @@ class PeerTubePlugin extends Plugin { | |||
217 | private playerElement: HTMLVideoElement | 217 | private playerElement: HTMLVideoElement |
218 | private videoFiles: VideoFile[] | 218 | private videoFiles: VideoFile[] |
219 | private torrent: WebTorrent.Torrent | 219 | private torrent: WebTorrent.Torrent |
220 | private autoplay = false | ||
220 | 221 | ||
221 | constructor (player: videojs.Player, options: PeertubePluginOptions) { | 222 | constructor (player: videojs.Player, options: PeertubePluginOptions) { |
222 | super(player, options) | 223 | super(player, options) |
223 | 224 | ||
225 | // Fix canplay event on google chrome by disabling default videojs autoplay | ||
226 | this.autoplay = this.player.options_.autoplay | ||
227 | this.player.options_.autoplay = false | ||
228 | |||
224 | this.videoFiles = options.videoFiles | 229 | this.videoFiles = options.videoFiles |
225 | 230 | ||
226 | // Hack to "simulate" src link in video.js >= 6 | 231 | // Hack to "simulate" src link in video.js >= 6 |
@@ -281,6 +286,7 @@ class PeerTubePlugin extends Plugin { | |||
281 | 286 | ||
282 | this.renderer = renderer | 287 | this.renderer = renderer |
283 | if (!this.player.paused()) this.player.play().then(done) | 288 | if (!this.player.paused()) this.player.play().then(done) |
289 | else done() | ||
284 | }) | 290 | }) |
285 | }) | 291 | }) |
286 | 292 | ||
@@ -349,18 +355,12 @@ class PeerTubePlugin extends Plugin { | |||
349 | const webTorrentButton = new WebTorrentButton(this.player) | 355 | const webTorrentButton = new WebTorrentButton(this.player) |
350 | controlBar.webTorrent = controlBar.el().insertBefore(webTorrentButton.el(), controlBar.progressControl.el()) | 356 | controlBar.webTorrent = controlBar.el().insertBefore(webTorrentButton.el(), controlBar.progressControl.el()) |
351 | 357 | ||
352 | if (this.player.options_.autoplay === true) { | 358 | if (this.autoplay === true) { |
353 | this.updateVideoFile() | 359 | this.updateVideoFile(undefined, () => this.player.play()) |
354 | } else { | 360 | } else { |
355 | this.player.one('play', () => { | 361 | this.player.one('play', () => { |
356 | // On firefox, we need to wait to load the video before playing | 362 | this.player.pause() |
357 | if (navigator.userAgent.toLowerCase().indexOf('firefox') !== -1) { | 363 | this.updateVideoFile(undefined, () => this.player.play()) |
358 | this.player.pause() | ||
359 | this.updateVideoFile(undefined, () => this.player.play()) | ||
360 | return | ||
361 | } | ||
362 | |||
363 | this.updateVideoFile(undefined) | ||
364 | }) | 364 | }) |
365 | } | 365 | } |
366 | } | 366 | } |