clean up client less aggressively

Author: toddtarsi

packages/selenium-ide/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@seleniumhq/selenium-ide",
-  "version": "4.0.0-alpha.29",
+  "version": "4.0.0-alpha.30",
   "private": true,
   "description": "Selenium IDE electron app",
   "author": "Todd <[email protected]>",

packages/selenium-ide/src/browser/windows/PlaybackWindow/preload/find-select.ts

@@ -29,12 +29,16 @@ const init = () => {
     processHighlightCommand
   )
   window.addEventListener('beforeunload', () => {
-    window.sideAPI.recorder.onRequestSelectElement.removeListener(
-      processSelectionCommand
-    )
-    window.sideAPI.recorder.onHighlightElement.removeListener(
-      processHighlightCommand
-    )
+    try {
+      window.sideAPI.recorder.onRequestSelectElement.removeListener(
+        processSelectionCommand
+      )
+      window.sideAPI.recorder.onHighlightElement.removeListener(
+        processHighlightCommand
+      )
+    } catch (e) {
+      // ignore
+    }
   })
 }
 

packages/selenium-ide/src/browser/windows/PlaybackWindow/preload/recorder.ts

@@ -59,9 +59,13 @@ export default class Recorder {
       this.getFrameLocation
     )
     this.window.addEventListener('beforeunload', () => {
-      this.window.sideAPI.recorder.onFrameRecalculate.removeListener(
-        this.getFrameLocation
-      )
+      try {
+        this.window.sideAPI.recorder.onFrameRecalculate.removeListener(
+          this.getFrameLocation
+        )
+      } catch (e) {
+        // ignore
+      }
     })
     // @ts-expect-error
     this.recordingState = {}
@@ -244,7 +248,9 @@ export default class Recorder {
     callback: MutationCallback,
     config: any
   ) {
-    const observer = new MutationObserver(callback.bind(this)) as ExpandedMutationObserver
+    const observer = new MutationObserver(
+      callback.bind(this)
+    ) as ExpandedMutationObserver
     observer.observerName = observerName
     observer.config = config
     this.mutationObservers[observerName] = observer