From 24f8a122d52b1e91df56d86b194bf7560a039d2d Mon Sep 17 00:00:00 2001
From: Dave Justice <djustice@mozilla.com>
Date: Tue, 9 May 2017 14:07:53 -0400
Subject: [PATCH] Fix playback stutter on erratic user input (#805)

- fixes #626
---
 components/playback-control.js | 5 +++--
 components/player-view.js      | 5 +++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/components/playback-control.js b/components/playback-control.js
index eb31753..732739a 100644
--- a/components/playback-control.js
+++ b/components/playback-control.js
@@ -1,5 +1,6 @@
 const React = require('react');
 const cn = require('classnames');
+const debounce = require('lodash.debounce');
 const ReactTooltip = require('react-tooltip');
 const sendMetricsEvent = require('../client-lib/send-metrics-event');
 
@@ -22,10 +23,10 @@ module.exports = class PlaybackControl extends React.Component {
   render() {
     return (
         <div className={cn('playback-button', {hidden: !this.props.hovered && !this.props.minimized})}>
-        <a onClick={this.play.bind(this)} data-tip data-for='play'
+        <a onClick={debounce(this.play.bind(this), 100)} data-tip data-for='play'
           className={cn('play', {hidden: this.props.playing})} />
         <ReactTooltip id='play' effect='solid' place='right'>{this.props.strings.ttPlay}</ReactTooltip>
-        <a onClick={this.pause.bind(this)} data-tip data-for='pause'
+        <a onClick={debounce(this.pause.bind(this), 100)} data-tip data-for='pause'
           className={cn('pause', {hidden: !this.props.playing})} />
         <ReactTooltip id='pause' effect='solid' place='right'>{this.props.strings.ttPause}</ReactTooltip>
         </div>
diff --git a/components/player-view.js b/components/player-view.js
index 33f7607..022fda1 100644
--- a/components/player-view.js
+++ b/components/player-view.js
@@ -1,6 +1,7 @@
 const React = require('react');
 const cn = require('classnames');
 const keyboardJS = require('keyboardjs');
+const debounce = require('lodash.debounce');
 const ReactPlayer = require('react-player');
 
 const AudioCtrl = require('../client-lib/audio-ctrl');
@@ -241,7 +242,7 @@ module.exports = class Player extends React.Component {
 
     const visualEl = this.props.queue[0].error ? (<ErrorView {...this.props} countdown={this.state.errorCount} />) :
           this.props.queue[0].player === 'audio' ?
-          (<div id='audio-container' ref='audio-container' onClick={this.handleVideoClick.bind(this)}/>) :
+          (<div id='audio-container' ref='audio-container' onClick={debounce(this.handleVideoClick.bind(this), 100)}/>) :
           (<ReactPlayer {...this.props} url={this.props.queue[0].url} ref='player'
                         onPlay={this.onPlay.bind(this)}
                         onPause={this.onPause.bind(this)}
@@ -278,7 +279,7 @@ module.exports = class Player extends React.Component {
     return (<div className='video-wrapper'
                  onMouseEnter={this.enterPlayer.bind(this)}
                  onMouseLeave={this.leavePlayer.bind(this)}
-                 onClick={this.handleVideoClick.bind(this)}>
+                 onClick={debounce(this.handleVideoClick.bind(this), 100)}>
               {exited}
               <PrevTrackBtn {...this.props} hovered={this.state.hovered} />
               <NextTrackBtn {...this.props} nextTrack={this.nextTrack.bind(this)} hovered={this.state.hovered} />