aboutsummaryrefslogtreecommitdiffstats
path: root/components/script/dom/htmlscriptelement.rs
diff options
context:
space:
mode:
authorAnthony Ramine <n.oxyde@gmail.com>2016-11-09 08:27:35 +0100
committerAnthony Ramine <n.oxyde@gmail.com>2016-11-10 14:58:10 +0100
commit330263bf2c6b4e877d45dee3b4cbfac4a67d3d43 (patch)
tree67ef1d280e5145884831aa25ec03f108af1627b3 /components/script/dom/htmlscriptelement.rs
parent085c52dfbce125b9f777eadb1ed149e04548fc45 (diff)
downloadservo-330263bf2c6b4e877d45dee3b4cbfac4a67d3d43.tar.gz
servo-330263bf2c6b4e877d45dee3b4cbfac4a67d3d43.zip
Update html5ever to 0.10.1
Diffstat (limited to 'components/script/dom/htmlscriptelement.rs')
-rw-r--r--components/script/dom/htmlscriptelement.rs31
1 files changed, 16 insertions, 15 deletions
diff --git a/components/script/dom/htmlscriptelement.rs b/components/script/dom/htmlscriptelement.rs
index 3b2057e4338..e2d71144b2c 100644
--- a/components/script/dom/htmlscriptelement.rs
+++ b/components/script/dom/htmlscriptelement.rs
@@ -27,7 +27,6 @@ use dom::node::{document_from_node, window_from_node};
use dom::virtualmethods::VirtualMethods;
use encoding::label::encoding_from_whatwg_label;
use encoding::types::{DecoderTrap, EncodingRef};
-use html5ever::tree_builder::NextParserState;
use html5ever_atoms::LocalName;
use ipc_channel::ipc;
use ipc_channel::router::ROUTER;
@@ -275,10 +274,12 @@ fn fetch_a_classic_script(script: &HTMLScriptElement,
impl HTMLScriptElement {
/// https://html.spec.whatwg.org/multipage/#prepare-a-script
- pub fn prepare(&self) -> NextParserState {
+ ///
+ /// Returns true if tokenization should continue, false otherwise.
+ pub fn prepare(&self) -> bool {
// Step 1.
if self.already_started.get() {
- return NextParserState::Continue;
+ return true;
}
// Step 2.
@@ -296,17 +297,17 @@ impl HTMLScriptElement {
// Step 4.
let text = self.Text();
if text.is_empty() && !element.has_attribute(&local_name!("src")) {
- return NextParserState::Continue;
+ return true;
}
// Step 5.
if !self.upcast::<Node>().is_in_doc() {
- return NextParserState::Continue;
+ return true;
}
// Step 6.
if !self.is_javascript() {
- return NextParserState::Continue;
+ return true;
}
// Step 7.
@@ -321,12 +322,12 @@ impl HTMLScriptElement {
// Step 9.
let doc = document_from_node(self);
if self.parser_inserted.get() && &*self.parser_document != &*doc {
- return NextParserState::Continue;
+ return true;
}
// Step 10.
if !doc.is_scripting_enabled() {
- return NextParserState::Continue;
+ return true;
}
// TODO(#4577): Step 11: CSP.
@@ -339,13 +340,13 @@ impl HTMLScriptElement {
let for_value = for_attribute.value().to_ascii_lowercase();
let for_value = for_value.trim_matches(HTML_SPACE_CHARACTERS);
if for_value != "window" {
- return NextParserState::Continue;
+ return true;
}
let event_value = event_attribute.value().to_ascii_lowercase();
let event_value = event_value.trim_matches(HTML_SPACE_CHARACTERS);
if event_value != "onload" && event_value != "onload()" {
- return NextParserState::Continue;
+ return true;
}
},
(_, _) => (),
@@ -380,7 +381,7 @@ impl HTMLScriptElement {
// Step 18.2.
if src.is_empty() {
self.queue_error_event();
- return NextParserState::Continue;
+ return true;
}
// Step 18.4-18.5.
@@ -388,7 +389,7 @@ impl HTMLScriptElement {
Err(_) => {
warn!("error parsing URL for script {}", &**src);
self.queue_error_event();
- return NextParserState::Continue;
+ return true;
}
Ok(url) => url,
};
@@ -411,7 +412,7 @@ impl HTMLScriptElement {
!async {
doc.add_deferred_script(self);
// Second part implemented in Document::process_deferred_scripts.
- return NextParserState::Continue;
+ return true;
// Step 20.b: classic, has src, was parser-inserted, is not async.
} else if is_external &&
was_parser_inserted &&
@@ -442,7 +443,7 @@ impl HTMLScriptElement {
self.ready_to_be_parser_executed.set(true);
*self.load.borrow_mut() = Some(Ok(ScriptOrigin::internal(text, base_url)));
self.execute();
- return NextParserState::Continue;
+ return true;
}
// TODO: make this suspension happen automatically.
@@ -451,7 +452,7 @@ impl HTMLScriptElement {
parser.suspend();
}
}
- NextParserState::Suspend
+ false
}
pub fn is_ready_to_be_executed(&self) -> bool {