jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/370832 )

Change subject: Fix 'seeking' message in ogv.js on end of audio file
......................................................................


Fix 'seeking' message in ogv.js on end of audio file

Use the 'seeked' event for end and don't trigger seeking mode
when not seeking.

Bug: T172888
Change-Id: I46bf5d4041ebaad648ca07e9e026fdd6699f4884
---
M MwEmbedModules/EmbedPlayer/resources/mw.EmbedPlayerOgvJs.js
1 file changed, 10 insertions(+), 5 deletions(-)

Approvals:
  Brion VIBBER: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/MwEmbedModules/EmbedPlayer/resources/mw.EmbedPlayerOgvJs.js 
b/MwEmbedModules/EmbedPlayer/resources/mw.EmbedPlayerOgvJs.js
index d523f99..44f3371 100644
--- a/MwEmbedModules/EmbedPlayer/resources/mw.EmbedPlayerOgvJs.js
+++ b/MwEmbedModules/EmbedPlayer/resources/mw.EmbedPlayerOgvJs.js
@@ -197,12 +197,17 @@
                this.currentTime = time;
                this.previousTime = time; // prevent weird double-seek. 
MwEmbedPlyer is weird!
 
-               // Run the onSeeking interface update
-               this.controlBuilder.onSeek();
-               // @todo add proper events upstream
                if( this.seeking ){
-                       this.seeking = false;
-                       $( this ).trigger( 'seeked' );
+                       // Run the onSeeking interface update
+                       this.controlBuilder.onSeek();
+                       var _this = this;
+                       function onseeked(event) {
+                               _this.seeking = false;
+                               _this.hideSpinner();
+                               $( _this ).trigger( 'seeked' );
+                               this.removeEventListener( 'seeked', onseeked );
+                       }
+                       this.playerElement.addEventListener( 'seeked', onseeked 
);
                }
                if ( $.isFunction( callback ) ) {
                        callback();

-- 
To view, visit https://gerrit.wikimedia.org/r/370832
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I46bf5d4041ebaad648ca07e9e026fdd6699f4884
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TimedMediaHandler
Gerrit-Branch: master
Gerrit-Owner: Brion VIBBER <br...@wikimedia.org>
Gerrit-Reviewer: Brion VIBBER <br...@wikimedia.org>
Gerrit-Reviewer: TheDJ <hartman.w...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to