From 896c75555753ae4313e629c7c903fcafba16c6e8 Mon Sep 17 00:00:00 2001
From: MinhHNguyen <minhhnguyen95@gmail.com>
Date: Fri, 23 Jun 2017 16:07:32 -0700
Subject: [PATCH] Update: Remove 0.25x playback on media files (#182)

---
 src/lib/viewers/media/Settings.js             | 20 +++++++++----------
 .../viewers/media/__tests__/Settings-test.js  |  7 +++++--
 2 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/src/lib/viewers/media/Settings.js b/src/lib/viewers/media/Settings.js
index d871bf50e..9c7a013b0 100644
--- a/src/lib/viewers/media/Settings.js
+++ b/src/lib/viewers/media/Settings.js
@@ -5,6 +5,8 @@ import { addActivationListener, removeActivationListener, decodeKeydown, insertT
 import { ICON_ARROW_LEFT, ICON_ARROW_RIGHT, ICON_CHECK_MARK } from '../../icons/icons';
 import { CLASS_ELEM_KEYBOARD_FOCUS } from '../../constants';
 
+const TYPE_SPEED = 'speed';
+const TYPE_QUALITY = 'quality';
 const CLASS_SETTINGS = 'bp-media-settings';
 const CLASS_SETTINGS_SELECTED = 'bp-media-settings-selected';
 const CLASS_SETTINGS_OPEN = 'bp-media-settings-is-open';
@@ -12,7 +14,7 @@ const CLASS_SETTINGS_SUBTITLES_UNAVAILABLE = 'bp-media-settings-subtitles-unavai
 const CLASS_SETTINGS_SUBTITLES_ON = 'bp-media-settings-subtitles-on';
 const SELECTOR_SETTINGS_SUB_ITEM = '.bp-media-settings-sub-item';
 const SELECTOR_SETTINGS_VALUE = '.bp-media-settings-value';
-const MEDIA_SPEEDS = ['0.25', '0.5', '1.0', '1.25', '1.5', '2.0'];
+const MEDIA_SPEEDS = ['0.5', '1.0', '1.25', '1.5', '2.0'];
 
 const SETTINGS_TEMPLATE = `<div class="bp-media-settings">
     <div class="bp-media-settings-menu-main bp-media-settings-menu" role="menu">
@@ -37,10 +39,6 @@ const SETTINGS_TEMPLATE = `<div class="bp-media-settings">
             <div class="bp-media-settings-arrow">${ICON_ARROW_LEFT}</div>
             <div class="bp-media-settings-label" aria-label="${__('media_speed')}">${__('media_speed')}</div>
         </div>
-        <div class="bp-media-settings-sub-item" data-type="speed" data-value="0.25" tabindex="0" role="menuitemradio">
-            <div class="bp-media-settings-icon">${ICON_CHECK_MARK}</div>
-            <div class="bp-media-settings-value">0.25</div>
-        </div>
         <div class="bp-media-settings-sub-item" data-type="speed" data-value="0.5" tabindex="0" role="menuitemradio">
             <div class="bp-media-settings-icon">${ICON_CHECK_MARK}</div>
             <div class="bp-media-settings-value">0.5</div>
@@ -136,7 +134,7 @@ const SUBTITLES_SUBITEM_TEMPLATE = `<div class="bp-media-settings-sub-item" data
     /**
      * List of subtitles in the menu. The subtitles menu will be populated in this order
      *
-     * @property {array}
+     * @property {Array}
      */
     subtitles = [];
 
@@ -186,8 +184,8 @@ const SUBTITLES_SUBITEM_TEMPLATE = `<div class="bp-media-settings-sub-item" data
         const quality = cache.get('media-quality') || 'auto';
         const speed = cache.get('media-speed') || '1.0';
 
-        this.chooseOption('quality', quality);
-        this.chooseOption('speed', speed);
+        this.chooseOption(TYPE_QUALITY, quality);
+        this.chooseOption(TYPE_SPEED, speed);
     }
 
     /**
@@ -215,7 +213,7 @@ const SUBTITLES_SUBITEM_TEMPLATE = `<div class="bp-media-settings-sub-item" data
      * Getter for testing purposes
      *
      * @private
-     * @return {array}
+     * @return {Array}
      */
     getMediaSpeeds() {
         return MEDIA_SPEEDS;
@@ -230,7 +228,7 @@ const SUBTITLES_SUBITEM_TEMPLATE = `<div class="bp-media-settings-sub-item" data
         const current = parseFloat(cache.get('media-speed') || '1.0');
         const higherSpeeds = MEDIA_SPEEDS.filter((speed) => parseFloat(speed) > current);
         if (higherSpeeds.length > 0) {
-            this.chooseOption('speed', higherSpeeds[0]);
+            this.chooseOption(TYPE_SPEED, higherSpeeds[0]);
         }
     }
 
@@ -243,7 +241,7 @@ const SUBTITLES_SUBITEM_TEMPLATE = `<div class="bp-media-settings-sub-item" data
         const current = parseFloat(cache.get('media-speed') || '1.0');
         const lowerSpeeds = MEDIA_SPEEDS.filter((speed) => parseFloat(speed) < current);
         if (lowerSpeeds.length > 0) {
-            this.chooseOption('speed', lowerSpeeds[lowerSpeeds.length - 1]);
+            this.chooseOption(TYPE_SPEED, lowerSpeeds[lowerSpeeds.length - 1]);
         }
     }
 
diff --git a/src/lib/viewers/media/__tests__/Settings-test.js b/src/lib/viewers/media/__tests__/Settings-test.js
index 0b388fa9e..896fed1a7 100644
--- a/src/lib/viewers/media/__tests__/Settings-test.js
+++ b/src/lib/viewers/media/__tests__/Settings-test.js
@@ -73,10 +73,13 @@ describe('lib/viewers/media/Settings', () => {
         });
 
         it('should not decrease speed after min', () => {
-            settings.chooseOption('speed', '0.5');
+            const speedOptions = settings.getMediaSpeeds();
+            expect(speedOptions.length).to.be.above(0);
+
+            settings.chooseOption('speed', speedOptions[0]);
             settings.decreaseSpeed();
             const speed = cache.get('media-speed');
-            expect(speed).to.equal('0.25');
+            expect(speed).to.equal(speedOptions[0]);
         });
     });