From 7c769fe9fab07cf50dff20fd74f8f38db6195bfe Mon Sep 17 00:00:00 2001 From: Benedikt Meurer Date: Mon, 2 Nov 2020 14:36:43 +0100 Subject: [PATCH] [formatter] Utilize available CPU cores. Drive-by-fix: Use uppercase naming for constants as per the JavaScript style guide. Change-Id: I7b9adb7ec1067ab79319d79dd68668a53bf29f46 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2514437 Reviewed-by: Paul Lewis Commit-Queue: Paul Lewis Auto-Submit: Benedikt Meurer --- front_end/formatter/FormatterWorkerPool.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/front_end/formatter/FormatterWorkerPool.js b/front_end/formatter/FormatterWorkerPool.js index 44fc308256..49b613a3d5 100644 --- a/front_end/formatter/FormatterWorkerPool.js +++ b/front_end/formatter/FormatterWorkerPool.js @@ -4,7 +4,7 @@ import * as Common from '../common/common.js'; -const MaxWorkers = 2; +const MAX_WORKERS = Math.min(2, navigator.hardwareConcurrency - 1); /** @type {!FormatterWorkerPool} */ let formatterWorkerPoolInstance; @@ -47,7 +47,7 @@ export class FormatterWorkerPool { } let freeWorker = [...this._workerTasks.keys()].find(worker => !this._workerTasks.get(worker)); - if (!freeWorker && this._workerTasks.size < MaxWorkers) { + if (!freeWorker && this._workerTasks.size < MAX_WORKERS) { freeWorker = this._createWorker(); } if (!freeWorker) {