Remove try/catch when importing node's worker_threads (#21045)

This seems like excessive error checking.  Node has supported
`worker_threads` going back v15 and the error folks will when this
module is missing should be pretty clear/actionable without us adding
this extra info.

Also remove unneeded local variable.

Author: sbc100

src/shell.js

@@ -265,14 +265,7 @@ if (ENVIRONMENT_IS_NODE) {
   Module['inspect'] = () => '[Emscripten Module object]';
 
 #if PTHREADS || WASM_WORKERS
-  let nodeWorkerThreads;
-  try {
-    nodeWorkerThreads = require('worker_threads');
-  } catch (e) {
-    console.error('The "worker_threads" module is not supported in this node.js build - perhaps a newer version is needed?');
-    throw e;
-  }
-  global.Worker = nodeWorkerThreads.Worker;
+  global.Worker = require('worker_threads').Worker;
 #endif
 
 #if WASM == 2

test/other/metadce/test_metadce_minimal_pthreads.gzsize

@@ -1 +1 @@
-5031
+4949

test/other/metadce/test_metadce_minimal_pthreads.jssize

@@ -1 +1 @@
-13863
+13710