Remove unused Window argument to onConnect callback

This argument was not used by any callers of `Bridge#onConnect` and
removing it eliminates a difference between creating a channel that uses
`Window.postMessage` vs one that uses `MessagePort`.

src/shared/bridge.js

@@ -14,7 +14,7 @@ export default class Bridge {
     this.links = [];
     /** @type {Record<string, (...args: any[]) => void>} */
     this.channelListeners = {};
-    /** @type {Array<(channel: RPC, window?: Window) => void>} */
+    /** @type {Array<(channel: RPC) => void>} */
     this.onConnectListeners = [];
   }
 
@@ -113,7 +113,7 @@ export default class Bridge {
         return;
       }
       connected = true;
-      this.onConnectListeners.forEach(cb => cb(channel, source));
+      this.onConnectListeners.forEach(cb => cb(channel));
     };
 
     const connect = (_token, cb) => {
@@ -227,7 +227,7 @@ export default class Bridge {
   /**
    * Add a listener to be called upon a new connection.
    *
-   * @param {(channel: RPC, window?: Window) => void} listener
+   * @param {(channel: RPC) => void} listener
    */
   onConnect(listener) {
     this.onConnectListeners.push(listener);

src/shared/test/bridge-test.js

@@ -258,7 +258,7 @@ describe('shared/bridge', () => {
 
       runConnectHandler(channel);
 
-      assert.calledWith(onConnectCallback, channel, fakeWindow);
+      assert.calledWith(onConnectCallback, channel);
     });
 
     it('does not run callback if a Window channel connects with wrong token', () => {
@@ -298,8 +298,8 @@ describe('shared/bridge', () => {
 
       runConnectHandler(channel);
 
-      assert.calledWith(onConnectCallback1, channel, fakeWindow);
-      assert.calledWith(onConnectCallback2, channel, fakeWindow);
+      assert.calledWith(onConnectCallback1, channel);
+      assert.calledWith(onConnectCallback2, channel);
     });
 
     it('only invokes `onConnect` callback once', () => {