foster/denoland-deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-11-21 15:04:11 -05:00
Code Issues Packages 1 Wiki Activity

refactor(runtime): "Worker::execute_script" returns value (#17092)

Browse source 
This commit changes "Worker::execute_script" to return a global
handle to "v8::Value".
This commit is contained in:
Fenix 2022-12-19 04:34:33 +08:00 committed by GitHub
parent 7b212bc574
commit 97f280eb9b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
runtime/worker.rs
View file

@ -13,6 +13,7 @@ use deno_core::error::AnyError;
use deno_core::error::JsError;
use deno_core::futures::Future;
use deno_core::located_script_name;
use deno_core::v8;
use deno_core::CompiledWasmModuleStore;
use deno_core::Extension;
use deno_core::FsModuleLoader;
@ -288,9 +289,8 @@ impl MainWorker {
&mut self,
script_name: &str,
source_code: &str,
) -> Result<(), AnyError> {
self.js_runtime.execute_script(script_name, source_code)?;
Ok(())
) -> Result<v8::Global<v8::Value>, AnyError> {
self.js_runtime.execute_script(script_name, source_code)
}
/// Loads and instantiates specified JavaScript module as "main" module.
@ -431,7 +431,8 @@ impl MainWorker {
// it. Instead we're using global `dispatchEvent` function which will
// used a saved reference to global scope.
"dispatchEvent(new Event('load'))",
)
)?;
Ok(())
}
/// Dispatches "unload" event to the JavaScript runtime.
@ -447,7 +448,8 @@ impl MainWorker {
// it. Instead we're using global `dispatchEvent` function which will
// used a saved reference to global scope.
"dispatchEvent(new Event('unload'))",
)
)?;
Ok(())
}
/// Dispatches "beforeunload" event to the JavaScript runtime. Returns a boolean