From 7a8bb3b611f02b272b1c19b6f3d8a85b099ca317 Mon Sep 17 00:00:00 2001 From: Nayeem Rahman Date: Thu, 4 May 2023 01:44:59 +0100 Subject: fix(core): allow esm extensions not included in snapshot (#18980) Fixes #18979. This changes the predicate for allowing `ext:` specifier resolution from `snapshot_loaded_and_not_snapshotting` to `ext_resolution_allowed` which is only set to true during the extension module loading phase. Module loaders as used in core are now declared as `ExtModuleLoader` rather than `dyn ModuleLoader`. --- bench_util/js_runtime.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'bench_util') diff --git a/bench_util/js_runtime.rs b/bench_util/js_runtime.rs index 57085ef96..a12e6ca62 100644 --- a/bench_util/js_runtime.rs +++ b/bench_util/js_runtime.rs @@ -10,9 +10,7 @@ use crate::profiling::is_profiling; pub fn create_js_runtime(setup: impl FnOnce() -> Vec) -> JsRuntime { JsRuntime::new(RuntimeOptions { extensions: setup(), - module_loader: Some( - std::rc::Rc::new(deno_core::ExtModuleLoader::default()), - ), + module_loader: None, ..Default::default() }) } -- cgit v1.2.3