diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-07-16 03:47:27 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-07-16 03:47:27 -0700 |
commit | e8fa02a07f3ee8965c022df4e92d5bdb0fccdb0b (patch) | |
tree | 3c7bc0b3f505fa0e0f73615e7dc2ca515efeae72 /components/script/dom | |
parent | d9b7c060a5ec7a6c0556de371b3e9ed3dc832337 (diff) | |
parent | b65240d730ade257f931149b4cc6dab0be6d8191 (diff) | |
download | servo-e8fa02a07f3ee8965c022df4e92d5bdb0fccdb0b.tar.gz servo-e8fa02a07f3ee8965c022df4e92d5bdb0fccdb0b.zip |
Auto merge of #12471 - Ms2ger:script-exec, r=nox
Cleanup HTMLScriptElement::execute()
<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12471)
<!-- Reviewable:end -->
Diffstat (limited to 'components/script/dom')
-rw-r--r-- | components/script/dom/htmlscriptelement.rs | 85 |
1 files changed, 48 insertions, 37 deletions
diff --git a/components/script/dom/htmlscriptelement.rs b/components/script/dom/htmlscriptelement.rs index 707e12ee59f..442f3063524 100644 --- a/components/script/dom/htmlscriptelement.rs +++ b/components/script/dom/htmlscriptelement.rs @@ -61,7 +61,7 @@ pub struct HTMLScriptElement { parser_document: JS<Document>, /// The source this script was loaded from - load: DOMRefCell<Option<ScriptOrigin>>, + load: DOMRefCell<Option<Result<ScriptOrigin, NetworkError>>>, } impl HTMLScriptElement { @@ -111,9 +111,28 @@ static SCRIPT_JS_MIMES: StaticStringVec = &[ ]; #[derive(HeapSizeOf, JSTraceable)] -pub enum ScriptOrigin { - Internal(DOMString, Url), - External(Result<(String, Url), NetworkError>), +pub struct ScriptOrigin { + text: DOMString, + url: Url, + external: bool, +} + +impl ScriptOrigin { + fn internal(text: DOMString, url: Url) -> ScriptOrigin { + ScriptOrigin { + text: text, + url: url, + external: false, + } + } + + fn external(text: DOMString, url: Url) -> ScriptOrigin { + ScriptOrigin { + text: text, + url: url, + external: true, + } + } } /// The context required for asynchronously loading an external script source. @@ -172,14 +191,14 @@ impl AsyncResponseListener for ScriptContext { // Step 7. let source_text = encoding.decode(&self.data, DecoderTrap::Replace).unwrap(); - (source_text, metadata.final_url) + ScriptOrigin::external(DOMString::from(source_text), metadata.final_url) }); // Step 9. // https://html.spec.whatwg.org/multipage/#prepare-a-script // Step 18.6 (When the chosen algorithm asynchronously completes). let elem = self.elem.root(); - *elem.load.borrow_mut() = Some(ScriptOrigin::External(load)); + *elem.load.borrow_mut() = Some(load); elem.ready_to_be_parser_executed.set(true); let document = document_from_node(elem.r()); @@ -374,13 +393,13 @@ impl HTMLScriptElement { // TODO: check for script nesting levels. doc.get_script_blocking_stylesheets_count() > 0 { doc.set_pending_parsing_blocking_script(Some(self)); - *self.load.borrow_mut() = Some(ScriptOrigin::Internal(text, base_url)); + *self.load.borrow_mut() = Some(Ok(ScriptOrigin::internal(text, base_url))); self.ready_to_be_parser_executed.set(true); // Step 20.f: otherwise. } else { assert!(!text.is_empty()); self.ready_to_be_parser_executed.set(true); - *self.load.borrow_mut() = Some(ScriptOrigin::Internal(text, base_url)); + *self.load.borrow_mut() = Some(Ok(ScriptOrigin::internal(text, base_url))); self.execute(); return NextParserState::Continue; } @@ -409,66 +428,58 @@ impl HTMLScriptElement { } let load = self.load.borrow_mut().take().unwrap(); - - // Step 2. - let (source, external, url) = match load { - // Step 2.a. - ScriptOrigin::External(Err(e)) => { + let script = match load { + // Step 2. + Err(e) => { warn!("error loading script {:?}", e); self.dispatch_error_event(); return; } - // Step 2.b.1.a. - ScriptOrigin::External(Ok((text, url))) => { - debug!("loading external script, url = {}", url); - (DOMString::from(text), true, url) - }, - - // Step 2.b.1.c. - ScriptOrigin::Internal(text, url) => { - (text, false, url) - } + Ok(script) => script, }; - // Step 2.b.2. + if script.external { + debug!("loading external script, url = {}", script.url); + } + + // TODO(#12446): beforescriptexecute. if !self.dispatch_before_script_execute_event() { return; } - // Step 2.b.3. + // Step 3. // TODO: If the script is from an external file, then increment the // ignore-destructive-writes counter of the script element's node // document. Let neutralised doc be that Document. - // Step 2.b.4. + // Step 4. let document = document_from_node(self); let document = document.r(); let old_script = document.GetCurrentScript(); - // Step 2.b.5. + // Step 5.a.1. document.set_current_script(Some(self)); - // Step 2.b.6. - // TODO: Create a script... + // Step 5.a.2. let window = window_from_node(self); rooted!(in(window.get_cx()) let mut rval = UndefinedValue()); - window.evaluate_script_on_global_with_result(&*source, - url.as_str(), - rval.handle_mut()); + window.evaluate_script_on_global_with_result(&script.text, + script.url.as_str(), + rval.handle_mut()); - // Step 2.b.7. + // Step 6. document.set_current_script(old_script.r()); - // Step 2.b.8. + // Step 7. // TODO: Decrement the ignore-destructive-writes counter of neutralised // doc, if it was incremented in the earlier step. - // Step 2.b.9. + // TODO(#12446): afterscriptexecute. self.dispatch_after_script_execute_event(); - // Step 2.b.10. - if external { + // Step 8. + if script.external { self.dispatch_load_event(); } else { window.dom_manipulation_task_source().queue_simple_event(self.upcast(), atom!("load"), window.r()); |