fix: fixes bug with multiple contentEndedListeners being registered

googleads · Oct 3, 2019 · 2208d86 · 2208d86
1 parent 317cdaa
commit 2208d86
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/player-wrapper.js b/src/player-wrapper.js
@@ -599,6 +599,11 @@ PlayerWrapper.prototype.addContentEndedListener = function(listener) {
  * Reset the player.
  */
 PlayerWrapper.prototype.reset = function() {
+  // Attempts to remove the contentEndedListener before adding it.
+  // This is to prevent an error where an erroring video caused multiple
+  // contentEndedListeners to be added.
+  this.vjsPlayer.off('contentended', this.boundContentEndedListener);
+
   this.vjsPlayer.on('contentended', this.boundContentEndedListener);
   this.vjsControls.show();
   if (this.vjsPlayer.ads.inAdBreak()) {