Enqueue upgrades

This commit is contained in:
Connor Brewster 2017-07-19 13:34:33 -06:00
parent 41371208a5
commit 6d9d4add61
12 changed files with 42 additions and 102 deletions

View file

@ -5,7 +5,7 @@
use dom::bindings::error::{report_pending_exception, throw_dom_exception}; use dom::bindings::error::{report_pending_exception, throw_dom_exception};
use dom::bindings::js::Root; use dom::bindings::js::Root;
use dom::bindings::reflector::DomObject; use dom::bindings::reflector::DomObject;
use dom::customelementregistry::is_valid_custom_element_name; use dom::customelementregistry::{is_valid_custom_element_name, upgrade_element};
use dom::document::Document; use dom::document::Document;
use dom::element::{CustomElementCreationMode, Element, ElementCreator}; use dom::element::{CustomElementCreationMode, Element, ElementCreator};
use dom::globalscope::GlobalScope; use dom::globalscope::GlobalScope;
@ -81,6 +81,7 @@ use dom::htmlvideoelement::HTMLVideoElement;
use dom::svgsvgelement::SVGSVGElement; use dom::svgsvgelement::SVGSVGElement;
use html5ever::{LocalName, Prefix, QualName}; use html5ever::{LocalName, Prefix, QualName};
use js::jsapi::JSAutoCompartment; use js::jsapi::JSAutoCompartment;
use script_thread::ScriptThread;
use servo_config::prefs::PREFS; use servo_config::prefs::PREFS;
fn create_svg_element(name: QualName, fn create_svg_element(name: QualName,
@ -127,8 +128,11 @@ fn create_html_element(name: QualName,
if let Some(definition) = definition { if let Some(definition) = definition {
if definition.is_autonomous() { if definition.is_autonomous() {
match mode { match mode {
// TODO: Handle asynchronous CE creation. Relies on CE upgrades. CustomElementCreationMode::Asynchronous => {
CustomElementCreationMode::Asynchronous => {}, let result = Root::upcast(HTMLElement::new(name.local.clone(), prefix.clone(), document));
ScriptThread::enqueue_upgrade_reaction(&*result, definition);
return result;
},
CustomElementCreationMode::Synchronous => { CustomElementCreationMode::Synchronous => {
let local_name = name.local.clone(); let local_name = name.local.clone();
return match definition.create_element(document, prefix.clone()) { return match definition.create_element(document, prefix.clone()) {
@ -155,9 +159,17 @@ fn create_html_element(name: QualName,
}, },
} }
} else { } else {
// Steps 5.1-5.2
let element = create_native_html_element(name, prefix, document, creator); let element = create_native_html_element(name, prefix, document, creator);
element.set_is(definition.name.clone()); element.set_is(definition.name.clone());
// TODO: Enqueue custom element upgrade match mode {
// Step 5.3
CustomElementCreationMode::Synchronous =>
upgrade_element(definition, &*element),
// Step 5.4
CustomElementCreationMode::Asynchronous =>
ScriptThread::enqueue_upgrade_reaction(&*element, definition),
}
return element; return element;
} }
} }

View file

@ -9,6 +9,7 @@ use dom::bindings::codegen::Bindings::CustomElementRegistryBinding::CustomElemen
use dom::bindings::codegen::Bindings::CustomElementRegistryBinding::ElementDefinitionOptions; use dom::bindings::codegen::Bindings::CustomElementRegistryBinding::ElementDefinitionOptions;
use dom::bindings::codegen::Bindings::ElementBinding::ElementMethods; use dom::bindings::codegen::Bindings::ElementBinding::ElementMethods;
use dom::bindings::codegen::Bindings::FunctionBinding::Function; use dom::bindings::codegen::Bindings::FunctionBinding::Function;
use dom::bindings::codegen::Bindings::WindowBinding::WindowBinding::WindowMethods;
use dom::bindings::conversions::{ConversionResult, FromJSValConvertible, StringificationBehavior}; use dom::bindings::conversions::{ConversionResult, FromJSValConvertible, StringificationBehavior};
use dom::bindings::error::{Error, ErrorResult, Fallible}; use dom::bindings::error::{Error, ErrorResult, Fallible};
use dom::bindings::inheritance::Castable; use dom::bindings::inheritance::Castable;
@ -281,17 +282,28 @@ impl CustomElementRegistryMethods for CustomElementRegistry {
self.element_definition_is_running.set(false); self.element_definition_is_running.set(false);
// Step 11 // Step 11
let definition = CustomElementDefinition::new(name.clone(), let definition = Rc::new(CustomElementDefinition::new(name.clone(),
local_name, local_name.clone(),
constructor_, constructor_,
observed_attributes, observed_attributes,
callbacks); callbacks));
// Step 12 // Step 12
self.definitions.borrow_mut().insert(name.clone(), Rc::new(definition)); self.definitions.borrow_mut().insert(name.clone(), definition.clone());
// TODO: Step 13, 14, 15 // Step 13
// Handle custom element upgrades let document = self.window.Document();
// Steps 14-15
for candidate in document.upcast::<Node>().traverse_preorder().filter_map(Root::downcast::<Element>) {
let is = candidate.get_is();
if *candidate.local_name() == local_name &&
*candidate.namespace() == ns!(html) &&
(extends.is_none() || is.as_ref() == Some(&name))
{
ScriptThread::enqueue_upgrade_reaction(&*candidate, definition.clone());
}
}
// Step 16, 16.3 // Step 16, 16.3
if let Some(promise) = self.when_defined.borrow_mut().remove(&name) { if let Some(promise) = self.when_defined.borrow_mut().remove(&name) {
@ -464,7 +476,7 @@ impl CustomElementDefinition {
/// https://html.spec.whatwg.org/multipage/#concept-upgrade-an-element /// https://html.spec.whatwg.org/multipage/#concept-upgrade-an-element
#[allow(unsafe_code)] #[allow(unsafe_code)]
fn upgrade_element(definition: Rc<CustomElementDefinition>, element: &Element) { pub fn upgrade_element(definition: Rc<CustomElementDefinition>, element: &Element) {
// TODO: Steps 1-2 // TODO: Steps 1-2
// Track custom element state // Track custom element state

View file

@ -30,7 +30,7 @@ use dom::bindings::str::{DOMString, USVString};
use dom::bindings::xmlname::namespace_from_domstring; use dom::bindings::xmlname::namespace_from_domstring;
use dom::characterdata::{CharacterData, LayoutCharacterDataHelpers}; use dom::characterdata::{CharacterData, LayoutCharacterDataHelpers};
use dom::cssstylesheet::CSSStyleSheet; use dom::cssstylesheet::CSSStyleSheet;
use dom::customelementregistry::CallbackReaction; use dom::customelementregistry::{CallbackReaction, try_upgrade_element};
use dom::document::{Document, DocumentSource, HasBrowsingContext, IsHTMLDocument}; use dom::document::{Document, DocumentSource, HasBrowsingContext, IsHTMLDocument};
use dom::documentfragment::DocumentFragment; use dom::documentfragment::DocumentFragment;
use dom::documenttype::DocumentType; use dom::documenttype::DocumentType;
@ -1639,12 +1639,13 @@ impl Node {
for descendant in kid.traverse_preorder().filter_map(Root::downcast::<Element>) { for descendant in kid.traverse_preorder().filter_map(Root::downcast::<Element>) {
// Step 7.7.2. // Step 7.7.2.
if descendant.is_connected() { if descendant.is_connected() {
// Step 7.7.2.1.
if descendant.get_custom_element_definition().is_some() { if descendant.get_custom_element_definition().is_some() {
// Step 7.7.2.1.
ScriptThread::enqueue_callback_reaction(&*descendant, CallbackReaction::Connected); ScriptThread::enqueue_callback_reaction(&*descendant, CallbackReaction::Connected);
} else {
// Step 7.7.2.2.
try_upgrade_element(&*descendant);
} }
// TODO: Step 7.7.2.2.
// Try to upgrade descendant.
} }
} }
} }

View file

@ -3,9 +3,6 @@
[HTML parser must not instantiate custom elements inside template elements] [HTML parser must not instantiate custom elements inside template elements]
expected: FAIL expected: FAIL
[HTML parser must use the registry of the content document inside an iframe]
expected: FAIL
[HTML parser must use the registry of window.document in a document created by document.implementation.createHTMLDocument()] [HTML parser must use the registry of window.document in a document created by document.implementation.createHTMLDocument()]
expected: FAIL expected: FAIL

View file

@ -1,5 +0,0 @@
[reaction-timing.html]
type: testharness
[Calling Node.prototype.cloneNode(false) must push a new element queue to the processing stack]
expected: FAIL

View file

@ -1,8 +1,5 @@
[Document.html] [Document.html]
type: testharness type: testharness
[importNode on Document must construct a new custom element when importing a custom element from a template]
expected: FAIL
[execCommand on Document must enqueue a disconnected reaction when deleting a custom element from a contenteditable element] [execCommand on Document must enqueue a disconnected reaction when deleting a custom element from a contenteditable element]
expected: FAIL expected: FAIL

View file

@ -3,9 +3,6 @@
[cloneNode on Node must enqueue an attributeChanged reaction when cloning an element with an observed attribute] [cloneNode on Node must enqueue an attributeChanged reaction when cloning an element with an observed attribute]
expected: FAIL expected: FAIL
[cloneNode on Node must not enqueue an attributeChanged reaction when cloning an element with an unobserved attribute]
expected: FAIL
[cloneNode on Node must enqueue an attributeChanged reaction when cloning an element only for observed attributes] [cloneNode on Node must enqueue an attributeChanged reaction when cloning an element only for observed attributes]
expected: FAIL expected: FAIL

View file

@ -3,9 +3,6 @@
[cloneContents on Range must enqueue an attributeChanged reaction when cloning an element with an observed attribute] [cloneContents on Range must enqueue an attributeChanged reaction when cloning an element with an observed attribute]
expected: FAIL expected: FAIL
[cloneContents on Range must not enqueue an attributeChanged reaction when cloning an element with an unobserved attribute]
expected: FAIL
[cloneContents on Range must enqueue an attributeChanged reaction when cloning an element only for observed attributes] [cloneContents on Range must enqueue an attributeChanged reaction when cloning an element only for observed attributes]
expected: FAIL expected: FAIL

View file

@ -1,50 +0,0 @@
[upgrading.html]
type: testharness
[Creating an element in the document of the template elements and inserting into the document must not enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in the document of the template elements and adopting back to a document with browsing context must enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in a new document and inserting into the document must not enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in a new document and adopting back to a document with browsing context must enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in a cloned document and inserting into the document must not enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in a cloned document and adopting back to a document with browsing context must enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in a document created by createHTMLDocument and inserting into the document must not enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in a document created by createHTMLDocument and adopting back to a document with browsing context must enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in an HTML document created by createDocument and inserting into the document must not enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in an HTML document created by createDocument and adopting back to a document with browsing context must enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in an HTML document fetched by XHR and inserting into the document must not enqueue a custom element upgrade reaction]
expected: FAIL
[Creating an element in an HTML document fetched by XHR and adopting back to a document with browsing context must enqueue a custom element upgrade reaction]
expected: FAIL
["define" in the document of an iframe must not enqueue a custom element upgrade reaction on a disconnected unresolved custom element]
expected: FAIL
[Inserting an unresolved custom element into the document of an iframe must enqueue a custom element upgrade reaction]
expected: FAIL
["define" in the document of an iframe must enqueue a custom element upgrade reaction on a connected unresolved custom element]
expected: FAIL
[Adopting and inserting an unresolved custom element into the document of an iframe must enqueue a custom element upgrade reaction]
expected: FAIL

View file

@ -1,17 +1,5 @@
[Node-cloneNode.html] [Node-cloneNode.html]
type: testharness type: testharness
[Node.prototype.cloneNode(false) must be able to clone a custom element]
expected: FAIL
[Node.prototype.cloneNode(false) must be able to clone a custom element inside an iframe]
expected: FAIL
[Node.prototype.cloneNode(true) must be able to clone a descendent custom element]
expected: FAIL
[Node.prototype.cloneNode(true) must set parentNode, previousSibling, and nextSibling before upgrading custom elements]
expected: FAIL
[HTMLElement constructor must throw an InvalidStateError when the top of the construction stack is marked AlreadyConstructed due to a custom element constructor constructing itself after super() call] [HTMLElement constructor must throw an InvalidStateError when the top of the construction stack is marked AlreadyConstructed due to a custom element constructor constructing itself after super() call]
expected: FAIL expected: FAIL

View file

@ -12,6 +12,3 @@
[Upgrading a custom element must enqueue attributeChangedCallback before connectedCallback] [Upgrading a custom element must enqueue attributeChangedCallback before connectedCallback]
expected: FAIL expected: FAIL
[Upgrading a custom element must not invoke attributeChangedCallback and connectedCallback when the element failed to upgrade]
expected: FAIL

View file

@ -1,8 +1,5 @@
[upgrading-parser-created-element.html] [upgrading-parser-created-element.html]
type: testharness type: testharness
[Element.prototype.createElement must add an unresolved custom element to the upgrade candidates map]
expected: FAIL
[HTMLElement constructor must throw an InvalidStateError when the top of the construction stack is marked AlreadyConstructed due to a custom element constructor constructing itself after super() call] [HTMLElement constructor must throw an InvalidStateError when the top of the construction stack is marked AlreadyConstructed due to a custom element constructor constructing itself after super() call]
expected: FAIL expected: FAIL