Preserve error name across web worker boundary so rejections keep DOMException type

Author: ljoy913

dist/msrcrypto.js

@@ -900,7 +900,19 @@ if (runningInWorkerInstance) {
             return msrcryptoWorker.returnResult({ initialized: true });
         }
 
-        if (workerInitialized === true) { msrcryptoWorker.jsCryptoRunner(e); }
+        if (workerInitialized === true) {
+            try {
+                msrcryptoWorker.jsCryptoRunner(e);
+            } catch (ex) {
+                msrcryptoWorker.returnResult({
+                    error: {
+                        name: (ex && ex.name) || "OperationError",
+                        message: (ex && ex.message) || "",
+                        code: (ex && ex.code) || 0
+                    }
+                });
+            }
+        }
 
     };
 }
@@ -9295,6 +9307,15 @@ var workerManager = (function() {
 
             e.target || (e.target = { data: worker.data });
 
+            if (e.data.error) {
+                jobCompleted(worker);
+                op.dispatchEvent({
+                    type: "error",
+                    data: utils.error(e.data.error.name, e.data.error.message)
+                });
+                return;
+            }
+
             for (var i = 0; i < jobQueue.length; i++) {
                 if (jobQueue[i].operation === worker.operation) {
                     var job = jobQueue[i];

dist/msrcrypto.min.js

@@ -226,6 +226,19 @@ var workerManager = (function() {
             // tslint:disable-next-line: no-unused-expression
             e.target || (e.target = { data: worker.data });
 
+            // A thrown error inside the worker is posted back as a structured
+            // envelope (a native ErrorEvent loses the original name/code).
+            // Rebuild a proper DOMException and reject this operation, mirroring
+            // the synchronous syncWorker error path.
+            if (e.data.error) {
+                jobCompleted(worker);
+                op.dispatchEvent({
+                    type: "error",
+                    data: utils.error(e.data.error.name, e.data.error.message)
+                });
+                return;
+            }
+
             // Check if there are queued jobs for this operation
             for (var i = 0; i < jobQueue.length; i++) {
                 if (jobQueue[i].operation === worker.operation) {

src/subtle/workerManager.js

@@ -106,7 +106,24 @@ if (runningInWorkerInstance) {
         }
 
         // Process the crypto operation
-        if (workerInitialized === true) { msrcryptoWorker.jsCryptoRunner(e); }
+        if (workerInitialized === true) {
+            try {
+                msrcryptoWorker.jsCryptoRunner(e);
+            } catch (ex) {
+                // A real web worker surfaces a thrown error as an ErrorEvent on
+                // the main thread, stripped of the original name/code (and it
+                // also bubbles as an uncaught error). Post a serializable
+                // envelope instead so the worker manager can rebuild a proper
+                // DOMException, matching the synchronous syncWorker path.
+                msrcryptoWorker.returnResult({
+                    error: {
+                        name: (ex && ex.name) || "OperationError",
+                        message: (ex && ex.message) || "",
+                        code: (ex && ex.code) || 0
+                    }
+                });
+            }
+        }
 
     };
 }