wasm: improve handling of qljs_vscode_create_document crashes

Expose LintingCrashed from DocumentLinter, not ProcessCrashed. This
fixes false positives in our test suite, allowing us to enable more
tests for more coverage.

Author: strager

wasm/quick-lint-js.js

@@ -69,10 +69,19 @@ class DocumentLinter {
     assertEqual(this._state, DocumentLinterState.NO_PARSER);
     this._state = DocumentLinterState.CREATING_PARSER;
     this._parserPromise = (async () => {
-      let factory = await this._documentProcessManager._processFactoryPromise;
-      // TODO(strager): Reuse processes across documents.
-      let process = await factory.createProcessAsync();
-      let parser = await process.createDocumentForVSCodeAsync();
+      let parser;
+      try {
+        let factory = await this._documentProcessManager._processFactoryPromise;
+        // TODO(strager): Reuse processes across documents.
+        let process = await factory.createProcessAsync();
+        parser = await process.createDocumentForVSCodeAsync();
+      } catch (e) {
+        if (e instanceof ProcessCrashed) {
+          throw new LintingCrashed(e);
+        } else {
+          throw e;
+        }
+      }
 
       if (this._state === DocumentLinterState.DISPOSED) {
         parser.dispose();
@@ -101,7 +110,7 @@ class DocumentLinter {
         } catch (e) {
           if (e instanceof DocumentLinterDisposed) {
             // Ignore.
-          } else if (e instanceof ProcessCrashed) {
+          } else if (e instanceof LintingCrashed) {
             // Ignore.
           } else {
             throw e;

wasm/test-document.js

@@ -207,15 +207,11 @@ describe("DocumentLinter", () => {
         !crashedProcesses.has(process),
         "Should not use previously-crashed process"
       );
-      // TODO(strager): Figure out why qljs_vscode_create_document failures
-      // cause this test to fail.
-      if (functionName !== "qljs_vscode_create_document") {
-        let shouldCrash = rng.nextCoinFlip();
-        coinFlips.push(shouldCrash);
-        if (shouldCrash) {
-          crashedProcesses.add(process);
-          throw new ProcessCrashed("(injected fault)");
-        }
+      let shouldCrash = rng.nextCoinFlip();
+      coinFlips.push(shouldCrash);
+      if (shouldCrash) {
+        crashedProcesses.add(process);
+        throw new ProcessCrashed("(injected fault)");
       }
     };