Fix bug causing nonexistent windows to remain in workspace trees.

Previously, when registering window observers fails, the
`window.garbageCollect()` call would be a no-op because the window is
not in `allWindowsMap`. This would lead to such windows being left bound
to the workspace tree, since binding happens inside the window
initializer.

Author: rickyz

Sources/AppBundle/tree/MacWindow.swift

@@ -33,20 +33,21 @@ final class MacWindow: Window {
                 app
             )
             let window = MacWindow(id, app, axWindow, parent: data.parent, adaptiveWeight: data.adaptiveWeight, index: data.index)
+            allWindowsMap[id] = window
 
             if window.observe(refreshObs, kAXUIElementDestroyedNotification) &&
                 window.observe(refreshObs, kAXWindowDeminiaturizedNotification) &&
                 window.observe(refreshObs, kAXWindowMiniaturizedNotification) &&
                 window.observe(movedObs, kAXMovedNotification) &&
                 window.observe(resizedObs, kAXResizedNotification)
             {
-                allWindowsMap[id] = window
                 debugWindowsIfRecording(window)
                 if !restoreClosedWindowsCacheIfNeeded(newlyDetectedWindow: window) {
                     tryOnWindowDetected(window, startup: startup)
                 }
                 return window
             } else {
+                // Removes `window` from `allWindowsMap`.
                 window.garbageCollect(skipClosedWindowsCache: true)
                 return nil
             }