aboutsummaryrefslogtreecommitdiffstats
path: root/components/script/dom
diff options
context:
space:
mode:
authorbors-servo <lbergstrom+bors@mozilla.com>2016-07-16 03:47:27 -0700
committerGitHub <noreply@github.com>2016-07-16 03:47:27 -0700
commite8fa02a07f3ee8965c022df4e92d5bdb0fccdb0b (patch)
tree3c7bc0b3f505fa0e0f73615e7dc2ca515efeae72 /components/script/dom
parentd9b7c060a5ec7a6c0556de371b3e9ed3dc832337 (diff)
parentb65240d730ade257f931149b4cc6dab0be6d8191 (diff)
downloadservo-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.rs85
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());