Handle case where there are no windows and thus no sensible bounding box

Resolves Sentry issue WINAMP2-JS-PROD-GC

js/actionCreators/windows.ts

@@ -159,6 +159,11 @@ export function centerWindows(box: {
       windowsInfo.filter(w => getOpen(w.key))
     );
 
+    if (bounding == null) {
+      // There are no windows to center
+      return;
+    }
+
     const boxHeight = bounding.bottom - bounding.top;
     const boxWidth = bounding.right - bounding.left;
 
@@ -211,6 +216,10 @@ export function ensureWindowsAreOnScreen(): Thunk {
     const bounding = Utils.calculateBoundingBox(
       windowsInfo.filter(w => getOpen(w.key))
     );
+    if (bounding == null) {
+      // There are no windows visible, so there's no work to do.
+      return;
+    }
     const positions = Selectors.getWindowPositions(state);
 
     // Are we good?

js/utils.ts

@@ -336,20 +336,23 @@ export function objectFilter<V>(
   }, {});
 }
 
-export const calculateBoundingBox = (windows: WindowInfo[]) =>
-  windows
-    .map(w => ({
-      left: w.x,
-      top: w.y,
-      bottom: w.y + w.height,
-      right: w.x + w.width,
-    }))
-    .reduce((b, w) => ({
-      left: Math.min(b.left, w.left),
-      top: Math.min(b.top, w.top),
-      bottom: Math.max(b.bottom, w.bottom),
-      right: Math.max(b.right, w.right),
-    }));
+export const calculateBoundingBox = (windows: WindowInfo[]) => {
+  if (windows.length === 0) {
+    return null;
+  }
+  const windowSizes = windows.map(w => ({
+    left: w.x,
+    top: w.y,
+    bottom: w.y + w.height,
+    right: w.x + w.width,
+  }));
+  return windowSizes.reduce((b, w) => ({
+    left: Math.min(b.left, w.left),
+    top: Math.min(b.top, w.top),
+    bottom: Math.max(b.bottom, w.bottom),
+    right: Math.max(b.right, w.right),
+  }));
+};
 
 export function findLastIndex<T>(arr: T[], cb: (val: T) => boolean) {
   for (let i = arr.length - 1; i >= 0; i--) {