migrate the logic of findAndLocateExistedChromiumExecutable to worker to prevent ui being block

This commit is contained in:
geekgeekrun
2024-03-06 00:26:07 +08:00
parent 4b1e91bba4
commit 75924a4747
2 changed files with 28 additions and 8 deletions
@@ -24,11 +24,19 @@ export enum DOWNLOAD_ERROR_EXIT_CODE {
}
async function findAndLocateExistedChromiumExecutable(): Promise<BrowserInfo | null> {
const worker: Worker = new CheckAndLocateExistedChromiumExecutableWorker()
worker.on('message', () => {
debugger
return new Promise((resolve, reject) => {
const worker: Worker = new CheckAndLocateExistedChromiumExecutableWorker()
worker.once('message', (data) => {
if (data.type === 'RESULT') {
resolve(data.data)
}
})
worker.once('message', (data) => {
if (data.type === 'ERROR') {
reject(data.error)
}
})
})
// TODO:
};
export const getAnyAvailablePuppeteerExecutable = async (): Promise<BrowserInfo | null> => {
@@ -38,14 +46,12 @@ export const getAnyAvailablePuppeteerExecutable = async (): Promise<BrowserInfo
}
// find existed browser - the one maybe actively installed by user or ship with os like Edge on windows
try {
debugger
const existedOne = (await findAndLocateExistedChromiumExecutable())
await saveLastUsedAndAvailableBrowserInfo(existedOne)
// save its path
return existedOne
} catch (err) {
console.error(err)
debugger
console.log('no existed browser path found')
}
// find existed browser - the fallback one
@@ -1,4 +1,18 @@
import { parentPort } from 'node:worker_threads'
import findAndLocateExistedChromiumExecutable from '../flow/CHECK_AND_DOWNLOAD_DEPENDENCIES/check-and-locate-existed-chromium-executable'
;(async () => {
try {
const result = await findAndLocateExistedChromiumExecutable()
parentPort?.postMessage({
type: 'RESULT',
data: result
})
} catch (error) {
parentPort?.postMessage({
type: 'ERROR',
error
})
}
console.log(globalThis, findAndLocateExistedChromiumExecutable)
debugger
process.exit(0)
})()