Skip to content

E
edx-platform
Commit 86865a68 by Carlos Andrés Rocha
Options

Merge pull request #1507 from MITx/bugfix/valera/video_alpha 

Bugfix/valera/video alpha
parents bfcc0559 c7679ea6
Showing with 9 additions and 0 deletions
common/lib/xmodule/xmodule/js/src/videoalpha/display/html5_video.js
...@@ -221,6 +221,15 @@ this.HTML5Video = (function () { ...@@ -221,6 +221,15 @@ this.HTML5Video = (function () {
// and end playing at the specified end time. After it was paused, or when a seek operation happeded, // and end playing at the specified end time. After it was paused, or when a seek operation happeded,
// the starting time and ending time will reset to the beginning and the end of the video respectively. // the starting time and ending time will reset to the beginning and the end of the video respectively.
this.video.addEventListener('canplay', function () { this.video.addEventListener('canplay', function () {
// Because firefox triggers 'canplay' event every time when 'currentTime' property
// changes, we must make sure that this block of code runs only once. Otherwise,
// this will be an endless loop ('currentTime' property is changed below).
//
// Chrome is immune to this behavior.
if (_this.playerState !== HTML5Video.PlayerState.UNSTARTED) {
return;
}
_this.playerState = HTML5Video.PlayerState.PAUSED; _this.playerState = HTML5Video.PlayerState.PAUSED;
if (_this.start > _this.video.duration) { if (_this.start > _this.video.duration) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment