Video Player: Don't show the loop message in unexpected timing

ui/file_manager/video_player/js/media_controls.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 'use strict';
 
 /**
  * @fileoverview MediaControls class implements media playback controls
  * that exist outside of the audio/video HTML element.
  */
@@ skipping 1008 matching lines @@
   }.bind(this), 0);
 };
 
 /**
  * Shows a text banner.
  *
  * @param {string} identifier String identifier.
  * @private
  */
 VideoControls.prototype.showTextBanner_ = function(identifier) {
+  var text = this.stringFunction_(identifier);
+
   this.textBanner_.removeAttribute('visible');
-  this.textBanner_.textContent = this.stringFunction_(identifier);
+  this.textBanner_.textContent = text;
+
   setTimeout(function() {
+    var onAnimationEnd = function(text, event) {

[fukino, 2014/09/29 08:04:29]

This 'text' looks unused. Can we get rid of it?

[yoshiki, 2014/09/29 08:10:19]

Done.

+      this.textBanner_.removeEventListener(
+          'webkitAnimationEnd', onAnimationEnd);
+      this.textBanner_.removeAttribute('visible');
+    }.bind(this, text);
+    this.textBanner_.addEventListener('webkitAnimationEnd', onAnimationEnd);
+
     this.textBanner_.setAttribute('visible', 'true');
   }.bind(this), 0);
 };
 
 /**
  * Toggle play/pause state on a mouse click on the play/pause button. Can be
  * called externally.
  *
  * @param {Event} event Mouse click event.
  */
@@ skipping 185 matching lines @@
 AudioControls.prototype.onAdvanceClick_ = function(forward) {
   if (!forward &&
       (this.getMedia().currentTime > AudioControls.TRACK_RESTART_THRESHOLD)) {
     // We are far enough from the beginning of the current track.
     // Restart it instead of than skipping to the previous one.
     this.getMedia().currentTime = 0;
   } else {
     this.advanceTrack_(forward);
   }
 };