andywer / andywer/threads.js

threads in Legacy (Safari 7) browser

未關閉
#492 0 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
主要語言
TypeScript
星號
3.5k
分支
173
PR 合併指標
30 天內沒有已合併 PR

描述

Hey, Im trying to use threads (1.7.0) in a legacy device - Safari 7
The code works on modern GoogleChrome, but when trying to run it on Safari 7, it throws when importing the library.

The following code prints only `Before load threads` and then throws `Super expression must either be null or a function`

```
try {
console.error('Before load threads');
const { Worker } = await import('threads');
console.error('after load threads'); // never prints
} catch (error) {
console.error('Error loading threads', error); // throws ...."Super expression must either be null or a function"
}
```
Any idea what can it be or how to fix it?

PS: The issue is not the async import, for sure: the file where this code is written, is itself imported async. And just to be sure I tried also with regular import (at the head of the file), and got the same error.

**Update 1**: The same happens with Safari 7, 8 and 9. It works on Safari 10

**Update 2**: The cause for the exception is in the file `master/implementation.browser.js`, it is checking ` if (typeof Worker === "undefined" `.
For Safari 3 and below, `typeof Worker === "undefined". For Safari 10 and up, `typeof Worker === "function"`.
But for Safari 7-9, `typeof Worker === "object". So when it tries to extend from Worker on line 24 `class WebWorker extends Worker ` it fails because is an object, not a function and throws the `Super expression ...` exception

貢獻指南

這個儲存庫沒有索引到貢獻指南

評估

這個 Issue 還沒有評估資料。

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。