VideoPlayer: Remove unused AudioControl class.

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.
 
 /**
  * @fileoverview MediaControls class implements media playback controls
  * that exist outside of the audio/video HTML element.
  */
 
 /**
@@ skipping 227 matching lines @@
 /*
  * Time controls
  */
 
 /**
  * The default range of 100 is too coarse for the media progress slider.
  */
 MediaControls.PROGRESS_RANGE = 5000;
 
 /**
- * @param {boolean=} opt_seekMark True if the progress slider should have
- *     a seek mark.
  * @param {HTMLElement=} opt_parent Parent container.
  */
-MediaControls.prototype.initTimeControls = function(opt_seekMark, opt_parent) {

[fukino, 2015/10/15 12:45:05]

After removing AudioControl class, opt_seemMark is always true, so I removed it.

+MediaControls.prototype.initTimeControls = function(opt_parent) {
   var timeControls = this.createControl('time-controls', opt_parent);
 
-  var sliderConstructor =
-      opt_seekMark ? MediaControls.PreciseSlider : MediaControls.Slider;
-
-  this.progressSlider_ = new sliderConstructor(
+  this.progressSlider_ = new MediaControls.PreciseSlider(
       this.createControl('progress media-control', timeControls),
       0, /* value */
       MediaControls.PROGRESS_RANGE,
       this.onProgressChange_.bind(this),
       this.onProgressDrag_.bind(this));
 
   var timeBox = this.createControl('time media-control', timeControls);
 
   this.duration_ = this.createControl('duration', timeBox);
   // Set the initial width to the minimum to reduce the flicker.
@@ skipping 572 matching lines @@
 /**
  * Create a customized slider with a precise time feedback.
  *
  * The time value is shown above the slider bar at the mouse position.
  *
  * @param {!HTMLElement} container The containing div element.
  * @param {number} value Initial value [0..1].
  * @param {number} range Number of distinct slider positions to be supported.
  * @param {function(number)} onChange Value change handler.
  * @param {function(boolean)} onDrag Drag begin/end handler.
- * @param {function(number):string} formatFunction Value formatting function.
  * @constructor
  * @struct
  * @extends {MediaControls.Slider}
  */
 MediaControls.PreciseSlider = function(
-    container, value, range, onChange, onDrag, formatFunction) {

[fukino, 2015/10/15 12:45:05]

unused argument.

+    container, value, range, onChange, onDrag) {
   MediaControls.Slider.apply(this, arguments);
 
   var doc = this.container_.ownerDocument;
 
   /**
    * @type {number}
    * @private
    */
   this.latestMouseUpTime_ = 0;
 
@@ skipping 197 matching lines @@
  * @struct
  * @extends {MediaControls}
  */
 function VideoControls(containerElement, onMediaError, stringFunction,
     opt_fullScreenToggle, opt_stateIconParent) {
   MediaControls.call(this, containerElement, onMediaError);
   this.stringFunction_ = stringFunction;
 
   this.container_.classList.add('video-controls');
   this.initPlayButton();
-  this.initTimeControls(true /* show seek mark */);
+  this.initTimeControls();
   this.initVolumeControls();
 
   // Create the cast button.
   this.castButton_ = this.createButton('cast menubutton');
   this.castButton_.setAttribute('menu', '#cast-menu');
   this.castButton_.setAttribute(
       'label', this.stringFunction_('VIDEO_PLAYER_PLAY_ON'));
   cr.ui.decorate(this.castButton_, cr.ui.MenuButton);
 
   if (opt_fullScreenToggle) {
@@ skipping 207 matching lines @@
   var hideBelow = function(selector, limit) {
     this.container_.querySelector(selector).style.display =
         width < limit ? 'none' : '-webkit-box';
   }.bind(this);
 
   hideBelow('.time', 350);
   hideBelow('.volume', 275);
   hideBelow('.volume-controls', 210);
   hideBelow('.fullscreen', 150);
 };
-
-/**
- * Creates audio controls.
- *
- * @param {!HTMLElement} container Parent container.
- * @param {function(boolean)} advanceTrack Parameter: true=forward.
- * @param {function(Event)} onError Error handler.
- * @constructor
- * @struct
- * @extends {MediaControls}
- */
-function AudioControls(container, advanceTrack, onError) {
-  MediaControls.call(this, container, onError);
-
-  this.container_.classList.add('audio-controls');
-
-  this.advanceTrack_ = advanceTrack;
-
-  this.initPlayButton();
-  this.initTimeControls(false /* no seek mark */);
-  /* No volume controls */
-  this.createButton('previous', this.onAdvanceClick_.bind(this, false));
-  this.createButton('next', this.onAdvanceClick_.bind(this, true));
-
-  // Disables all controls at first.
-  this.enableControls_('.media-control', false);
-
-  var audioControls = this;
-  chrome.mediaPlayerPrivate.onNextTrack.addListener(
-      function() { audioControls.onAdvanceClick_(true); });
-  chrome.mediaPlayerPrivate.onPrevTrack.addListener(
-      function() { audioControls.onAdvanceClick_(false); });
-  chrome.mediaPlayerPrivate.onTogglePlayState.addListener(
-      function() { audioControls.togglePlayState(); });
-}
-
-AudioControls.prototype = { __proto__: MediaControls.prototype };
-
-/**
- * Media completion handler. Advances to the next track.
- */
-AudioControls.prototype.onMediaComplete = function() {
-  this.advanceTrack_(true);
-};
-
-/**
- * The track position after which "previous" button acts as "restart".
- */
-AudioControls.TRACK_RESTART_THRESHOLD = 5;  // seconds.
-
-/**
- * @param {boolean} forward True if advancing forward.
- * @private
- */
-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);
-  }
-};