From dd14e67c70cd7b5b6891295759cb08aa2f94f180 Mon Sep 17 00:00:00 2001
From: Florian Dold <florian@dold.me>
Date: Wed, 5 Oct 2022 12:52:49 +0200
Subject: wallet-core: improve crypto worker code duplication

Also add new testCrypto call for later testing
---
 packages/taler-wallet-webextension/src/browserCryptoWorkerFactory.ts | 2 ++
 packages/taler-wallet-webextension/src/wxBackend.ts                  | 2 ++
 2 files changed, 4 insertions(+)

(limited to 'packages/taler-wallet-webextension/src')

diff --git a/packages/taler-wallet-webextension/src/browserCryptoWorkerFactory.ts b/packages/taler-wallet-webextension/src/browserCryptoWorkerFactory.ts
index 75ec0906c..c93097da8 100644
--- a/packages/taler-wallet-webextension/src/browserCryptoWorkerFactory.ts
+++ b/packages/taler-wallet-webextension/src/browserCryptoWorkerFactory.ts
@@ -28,6 +28,8 @@ export class BrowserCryptoWorkerFactory implements CryptoWorkerFactory {
   startWorker(): CryptoWorker {
     const workerCtor = Worker;
     const workerPath = "/dist/browserWorkerEntry.js";
+    // FIXME: This is not really the same interface as the crypto worker!
+    // We need to wrap it.
     return new workerCtor(workerPath) as CryptoWorker;
   }
 
diff --git a/packages/taler-wallet-webextension/src/wxBackend.ts b/packages/taler-wallet-webextension/src/wxBackend.ts
index 9581abc56..68ec15591 100644
--- a/packages/taler-wallet-webextension/src/wxBackend.ts
+++ b/packages/taler-wallet-webextension/src/wxBackend.ts
@@ -217,6 +217,8 @@ async function reinitWallet(): Promise<void> {
     timer = new SetTimeoutTimerAPI();
   } else {
     httpLib = new BrowserHttpLib();
+    // We could (should?) use the BrowserCryptoWorkerFactory here,
+    // but right now we don't, to have less platform differences.
     cryptoWorker = new SynchronousCryptoWorkerFactory();
     timer = new SetTimeoutTimerAPI();
   }
-- 
cgit v1.2.3