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

make sure player callback has access to latest desktop start time

Browse source
This commit is contained in:
Sean Yesmunt 2020-05-05 16:45:59 -04:00
parent ab70f1c758
commit ff429117cd
2 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
ui/component/fileRenderFloating/view.jsx
View file

@ -78,6 +78,12 @@ export default function FileRenderFloating(props: Props) {
// @if TARGET='app' // @if TARGET='app'
setDesktopPlayStartTime(Date.now()); setDesktopPlayStartTime(Date.now());
// @endif // @endif
return () => {
// @if TARGET='app'
setDesktopPlayStartTime(undefined);
// @endif
};
}, [uri]); }, [uri]);
if (!isPlayable || !uri || (isFloating && (isMobile || !floatingPlayerEnabled))) { if (!isPlayable || !uri || (isFloating && (isMobile || !floatingPlayerEnabled))) {

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

@ -88,6 +88,7 @@ function VideoViewer(props: Props) {
function doTrackingFirstPlay(e: Event, data: any) { function doTrackingFirstPlay(e: Event, data: any) {
let timeToStartInMs = data.secondsToLoad * 1000; let timeToStartInMs = data.secondsToLoad * 1000;
if (desktopPlayStartTime !== undefined) { if (desktopPlayStartTime !== undefined) {
const differenceToAdd = Date.now() - desktopPlayStartTime; const differenceToAdd = Date.now() - desktopPlayStartTime;
timeToStartInMs += differenceToAdd; timeToStartInMs += differenceToAdd;
@ -166,7 +167,7 @@ function VideoViewer(props: Props) {
} }
player.on('dispose', () => savePosition(uri, player.currentTime())); player.on('dispose', () => savePosition(uri, player.currentTime()));
}, },
[uri] IS_WEB ? [uri] : [uri, desktopPlayStartTime]
); );
return ( return (