@raydiant/playlist-window-manager 0.28.2-rc5 → 0.29.0

package/build/PlaylistWindowManager.js

@@ -954,12 +954,12 @@ var PlaylistWindowManager = /** @class */ (function (_super) {
                                     switch (_a.label) {
                                         case 0:
                                             if (!windowToClose) return [3 /*break*/, 2];
-                                            // TODO: Do we need to check if the window is already removed/destroyed?
-                                            // TODO: Add a guard in BrowserWindowRenderer.removeWindow to check if the window is destroyed?
+                                            // We can't use this.removeCurrentWindow because the currentWindow reference has already
+                                            // changed by the time the timeout fires.
                                             return [4 /*yield*/, windowToClose.remove()];
                                         case 1:
-                                            // TODO: Do we need to check if the window is already removed/destroyed?
-                                            // TODO: Add a guard in BrowserWindowRenderer.removeWindow to check if the window is destroyed?
+                                            // We can't use this.removeCurrentWindow because the currentWindow reference has already
+                                            // changed by the time the timeout fires.
                                             _a.sent();
                                             _a.label = 2;
                                         case 2:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raydiant/playlist-window-manager",
-  "version": "0.28.2-rc5",
+  "version": "0.29.0",
   "description": "Window manager for playlist playback.",
   "main": "build/index.js",
   "scripts": {

package/src/PlaylistWindowManager.ts

@@ -749,8 +749,6 @@ export default class PlaylistWindowManager extends EventEmitter {
       this.nextWindow.show(this.isFocused);
     }
 
-    // If current window isn't defined we don't need to hide it but we still
-    // want to make the next window the new current window.
     // If current window isn't defined we don't need to hide it but we still
     // want to make the next window the new current window.
     let closeWindowPromise;
@@ -764,12 +762,9 @@ export default class PlaylistWindowManager extends EventEmitter {
           const windowToClose = this.currentWindow;
           setTimeout(async () => {
             if (windowToClose) {
-              // TODO: Do we need to check if the window is already removed/destroyed?
-              // TODO: Add a guard in BrowserWindowRenderer.removeWindow to check if the window is destroyed?
+              // We can't use this.removeCurrentWindow because the currentWindow reference has already
+              // changed by the time the timeout fires.
               await windowToClose.remove();
-              // TODO: We can't use this.removeCurrentWindow because the currentWindow reference has already
-              // changed by the time the timeout fires. Is there a better way to handle this?
-              // await this.removeCurrentWindow();
             }
             resolve(undefined);
           }, this.options.closeWindowDelayMs);