refactor "reset speed" functionality

This commit is contained in:
Ilya Grigorik
2017-05-25 20:46:54 -04:00
parent 735785fd06
commit 0f5f6c3aab

View File

@@ -312,13 +312,7 @@ chrome.extension.sendMessage({}, function(response) {
var s = Math.max(v.playbackRate - tc.settings.speedStep, 0.0625); var s = Math.max(v.playbackRate - tc.settings.speedStep, 0.0625);
v.playbackRate = Number(s.toFixed(2)); v.playbackRate = Number(s.toFixed(2));
} else if (action === 'reset') { } else if (action === 'reset') {
if(v.playbackRate === 1.0) { resetSpeed(v, 1.0);
v.playbackRate = tc.settings.resetSpeed;
} else {
tc.settings.resetSpeed = v.playbackRate;
chrome.storage.sync.set({'resetSpeed': v.playbackRate});
v.playbackRate = 1.0;
}
} else if (action === 'close') { } else if (action === 'close') {
v.classList.add('vsc-cancelled'); v.classList.add('vsc-cancelled');
controller.remove(); controller.remove();
@@ -328,19 +322,19 @@ chrome.extension.sendMessage({}, function(response) {
} else if (action === 'drag') { } else if (action === 'drag') {
handleDrag(v, controller); handleDrag(v, controller);
} else if (action === 'fast') { } else if (action === 'fast') {
playVideoAtFastSpeed(v); resetSpeed(v, tc.settings.fastSpeed);
} }
} }
}); });
} }
var oldSpeed = 1.0; function resetSpeed(v, target) {
function playVideoAtFastSpeed(video) { if (v.playbackRate === target) {
if(video.playbackRate == tc.settings.fastSpeed) { v.playbackRate = tc.settings.resetSpeed;
video.playbackRate = oldSpeed;
} else { } else {
oldSpeed = video.playbackRate; tc.settings.resetSpeed = v.playbackRate;
video.playbackRate = tc.settings.fastSpeed; chrome.storage.sync.set({'resetSpeed': v.playbackRate});
v.playbackRate = target;
} }
} }