From ed5a8fbd69ffa01e59ed3217ca4f144c153e2b24 Mon Sep 17 00:00:00 2001
From: nick-delirium <nikita@openreplay.com>
Date: Fri, 20 Jan 2023 09:28:42 +0100
Subject: [PATCH] change(ui): cleanup

---
 frontend/app/player/web/MessageManager.ts | 7 ++-----
 frontend/app/player/web/Screen/Screen.ts  | 4 +---
 frontend/app/player/web/WebPlayer.ts      | 6 +++---
 3 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/frontend/app/player/web/MessageManager.ts b/frontend/app/player/web/MessageManager.ts
index df6a91179..8c5c55bb2 100644
--- a/frontend/app/player/web/MessageManager.ts
+++ b/frontend/app/player/web/MessageManager.ts
@@ -119,8 +119,7 @@ export default class MessageManager {
     private readonly session: any /*Session*/,
     private readonly state: Store<State>,
     private readonly screen: Screen,
-    initialLists?: Partial<InitialLists>,
-    coldStart?: boolean
+    initialLists?: Partial<InitialLists>
   ) {
     this.pagesManager = new PagesManager(screen, this.session.isMobile, cssLoading => {
       screen.displayFrame(!cssLoading)
@@ -139,9 +138,7 @@ export default class MessageManager {
 
     this.activityManager = new ActivityManager(this.session.duration.milliseconds) // only if not-live
 
-    if (!coldStart) {
-      this.loadMessages()
-    }
+    void this.loadMessages()
   }
 
   private _sortMessagesHack(msgs: Message[]) {
diff --git a/frontend/app/player/web/Screen/Screen.ts b/frontend/app/player/web/Screen/Screen.ts
index 7fae818f6..0eb1d7245 100644
--- a/frontend/app/player/web/Screen/Screen.ts
+++ b/frontend/app/player/web/Screen/Screen.ts
@@ -57,9 +57,7 @@ export default class Screen {
   private readonly screen: HTMLDivElement;
   private parentElement: HTMLElement | null = null;
 
-  constructor(isMobile: boolean, coldStart?: boolean) {
-    if (coldStart) return;
-
+  constructor(isMobile: boolean) {
     const iframe = document.createElement('iframe');
     iframe.className = styles.iframe;
     this.iframe = iframe;
diff --git a/frontend/app/player/web/WebPlayer.ts b/frontend/app/player/web/WebPlayer.ts
index ac76701ad..81df4d3ca 100644
--- a/frontend/app/player/web/WebPlayer.ts
+++ b/frontend/app/player/web/WebPlayer.ts
@@ -25,7 +25,7 @@ export default class WebPlayer extends Player {
 
   private targetMarker: TargetMarker
 
-  constructor(protected wpState: Store<typeof WebPlayer.INITIAL_STATE>, session: any, live: boolean, coldStart?: boolean) {
+  constructor(protected wpState: Store<typeof WebPlayer.INITIAL_STATE>, session: any, live: boolean) {
     let initialLists = live ? {} : {
       event: session.events || [],
       stack: session.stackEvents || [],
@@ -40,8 +40,8 @@ export default class WebPlayer extends Player {
       ) || [],
     }
 
-    const screen = new Screen(session.isMobile, coldStart)
-    const messageManager = new MessageManager(session, wpState, screen, initialLists, coldStart)
+    const screen = new Screen(session.isMobile)
+    const messageManager = new MessageManager(session, wpState, screen, initialLists)
     super(wpState, messageManager)
     this.screen = screen
     this.messageManager = messageManager