LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Lint fixes

Browse source 
I just wanted to make the lint fixes in a separate commit to make the diff's for the next commit clearer.
This commit is contained in:
infiinte-persistence 2021-02-17 17:28:20 +08:00 committed by Jeremy Kauffman
parent 859cb5c645
commit 9ce6ecc923
1 changed files with 12 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
ui/component/viewers/videoViewer/view.jsx
View file

@ -19,8 +19,8 @@ const PLAY_TIMEOUT_LIMIT = 2000;
type Props = { type Props = {
position: number, position: number,
changeVolume: number => void, changeVolume: (number) => void,
changeMute: boolean => void, changeMute: (boolean) => void,
source: string, source: string,
contentType: string, contentType: string,
thumbnail: string, thumbnail: string,
@ -36,9 +36,9 @@ type Props = {
doAnalyticsBuffer: (string, any) => void, doAnalyticsBuffer: (string, any) => void,
claimRewards: () => void, claimRewards: () => void,
savePosition: (string, number) => void, savePosition: (string, number) => void,
clearPosition: string => void, clearPosition: (string) => void,
toggleVideoTheaterMode: () => void, toggleVideoTheaterMode: () => void,
setVideoPlaybackRate: number => void, setVideoPlaybackRate: (number) => void,
}; };
/* /*
@ -94,7 +94,7 @@ function VideoViewer(props: Props) {
}, [uri, previousUri]); }, [uri, previousUri]);
function doTrackingBuffered(e: Event, data: any) { function doTrackingBuffered(e: Event, data: any) {
fetch(source, { method: 'HEAD' }).then(response => { fetch(source, { method: 'HEAD' }).then((response) => {
data.playerPoweredBy = response.headers.get('x-powered-by'); data.playerPoweredBy = response.headers.get('x-powered-by');
doAnalyticsBuffer(uri, data); doAnalyticsBuffer(uri, data);
}); });
@ -150,12 +150,14 @@ function VideoViewer(props: Props) {
// https://blog.videojs.com/autoplay-best-practices-with-video-js/#Programmatic-Autoplay-and-Success-Failure-Detection // https://blog.videojs.com/autoplay-best-practices-with-video-js/#Programmatic-Autoplay-and-Success-Failure-Detection
if (shouldPlay) { if (shouldPlay) {
const playPromise = player.play(); const playPromise = player.play();
const timeoutPromise = new Promise((resolve, reject) => setTimeout(() => reject(PLAY_TIMEOUT_ERROR), PLAY_TIMEOUT_LIMIT)); const timeoutPromise = new Promise((resolve, reject) =>
setTimeout(() => reject(PLAY_TIMEOUT_ERROR), PLAY_TIMEOUT_LIMIT)
);
Promise.race([playPromise, timeoutPromise]).catch(error => { Promise.race([playPromise, timeoutPromise]).catch((error) => {
if (PLAY_TIMEOUT_ERROR) { if (PLAY_TIMEOUT_ERROR) {
const retryPlayPromise = player.play(); const retryPlayPromise = player.play();
Promise.race([retryPlayPromise, timeoutPromise]).catch(error => { Promise.race([retryPlayPromise, timeoutPromise]).catch((error) => {
setIsLoading(false); setIsLoading(false);
setIsPlaying(false); setIsPlaying(false);
}); });
@ -167,6 +169,7 @@ function VideoViewer(props: Props) {
} }
setIsLoading(shouldPlay); // if we are here outside of an embed, we're playing setIsLoading(shouldPlay); // if we are here outside of an embed, we're playing
player.on('tracking:buffered', doTrackingBuffered); player.on('tracking:buffered', doTrackingBuffered);
player.on('tracking:firstplay', doTrackingFirstPlay); player.on('tracking:firstplay', doTrackingFirstPlay);
player.on('ended', onEnded); player.on('ended', onEnded);
@ -175,9 +178,8 @@ function VideoViewer(props: Props) {
setIsPlaying(false); setIsPlaying(false);
handlePosition(player); handlePosition(player);
}); });
player.on('error', function() { player.on('error', () => {
const error = player.error(); const error = player.error();
if (error) { if (error) {
analytics.sentryError('Video.js error', error); analytics.sentryError('Video.js error', error);
} }