Move vscodeMessageHandler to before it is used

Author: robertbrignull

extensions/ql-vscode/src/view/results/ResultTables.tsx

@@ -307,6 +307,14 @@ export class ResultTables extends React.Component<
     }
   }
 
+  private vscodeMessageHandler(evt: MessageEvent) {
+    // sanitize origin
+    const origin = evt.origin.replace(/\n|\r/g, "");
+    evt.origin === window.origin
+      ? this.handleMessage(evt.data as IntoResultsViewMsg)
+      : console.error(`Invalid event origin ${origin}`);
+  }
+
   // TODO: Duplicated from results.tsx consider a way to
   // avoid this duplication
   componentDidMount(): void {
@@ -319,14 +327,6 @@ export class ResultTables extends React.Component<
       window.removeEventListener("message", this.vscodeMessageHandler);
     }
   }
-
-  private vscodeMessageHandler(evt: MessageEvent) {
-    // sanitize origin
-    const origin = evt.origin.replace(/\n|\r/g, "");
-    evt.origin === window.origin
-      ? this.handleMessage(evt.data as IntoResultsViewMsg)
-      : console.error(`Invalid event origin ${origin}`);
-  }
 }
 
 function getDefaultResultSet(resultSets: readonly ResultSet[]): string {