diff --git a/components/script/dom/range.rs b/components/script/dom/range.rs index 235a4dbcc9b..7d94d95f0e4 100644 --- a/components/script/dom/range.rs +++ b/components/script/dom/range.rs @@ -118,7 +118,7 @@ impl Range { // Step 11. let contained_children: Vec> = - common_ancestor.children().filter(|n| Range::contains(self, n)).collect(); + common_ancestor.children().filter(|n| self.contains(n)).collect(); // Step 12. if contained_children.iter().any(|n| n.is_doctype()) { @@ -850,6 +850,49 @@ impl RangeMethods for Range { // Step 7. self.SelectNode(new_parent) } + + // https://dom.spec.whatwg.org/#dom-range-stringifier + fn Stringifier(&self) -> DOMString { + let start_node = self.StartContainer(); + let end_node = self.EndContainer(); + + // Step 1. + let mut s = DOMString::new(); + + if let Some(text_node) = start_node.downcast::() { + let char_data = text_node.upcast::(); + + // Step 2. + if start_node == end_node { + return char_data.SubstringData(self.StartOffset(), + self.EndOffset() - self.StartOffset()).unwrap(); + } + + // Step 3. + s.push_str(&*char_data.SubstringData(self.StartOffset(), + char_data.Length() - self.StartOffset()).unwrap()); + } + + // Step 4. + let ancestor = self.CommonAncestorContainer(); + let mut iter = start_node.following_nodes(ancestor.r()) + .filter_map(Root::downcast::); + + while let Some(child) = iter.next() { + if self.contains(child.upcast()) { + s.push_str(&*child.upcast::().Data()); + } + } + + // Step 5. + if let Some(text_node) = end_node.downcast::() { + let char_data = text_node.upcast::(); + s.push_str(&*char_data.SubstringData(0, self.EndOffset()).unwrap()); + } + + // Step 6. + s + } } #[derive(JSTraceable)] diff --git a/components/script/dom/webidls/Range.webidl b/components/script/dom/webidls/Range.webidl index 1d91c89b505..4c7413e3456 100644 --- a/components/script/dom/webidls/Range.webidl +++ b/components/script/dom/webidls/Range.webidl @@ -72,7 +72,7 @@ interface Range { [Pure] boolean intersectsNode(Node node); - // stringifier; + stringifier; }; // https://dvcs.w3.org/hg/innerhtml/raw-file/tip/index.html#extensions-to-the-range-interface diff --git a/tests/wpt/metadata/MANIFEST.json b/tests/wpt/metadata/MANIFEST.json index 98aaa6d001e..cef25a30532 100644 --- a/tests/wpt/metadata/MANIFEST.json +++ b/tests/wpt/metadata/MANIFEST.json @@ -34520,6 +34520,12 @@ "url": "/dom/lists/DOMTokenList-value.html" } ], + "dom/ranges/Range-stringifier.html": [ + { + "path": "dom/ranges/Range-stringifier.html", + "url": "/dom/ranges/Range-stringifier.html" + } + ], "html/webappapis/scripting/events/event-handler-processing-algorithm.html": [ { "path": "html/webappapis/scripting/events/event-handler-processing-algorithm.html", diff --git a/tests/wpt/metadata/dom/interfaces.html.ini b/tests/wpt/metadata/dom/interfaces.html.ini index c9c0283d992..1d426dc5fdd 100644 --- a/tests/wpt/metadata/dom/interfaces.html.ini +++ b/tests/wpt/metadata/dom/interfaces.html.ini @@ -129,9 +129,6 @@ [Element interface: calling queryAll(DOMString) on element with too few arguments must throw TypeError] expected: FAIL - [Range interface: stringifier] - expected: FAIL - [DOMSettableTokenList interface: existence and properties of interface prototype object] expected: FAIL diff --git a/tests/wpt/web-platform-tests/dom/ranges/Range-stringifier.html b/tests/wpt/web-platform-tests/dom/ranges/Range-stringifier.html new file mode 100644 index 00000000000..330c7421ea1 --- /dev/null +++ b/tests/wpt/web-platform-tests/dom/ranges/Range-stringifier.html @@ -0,0 +1,44 @@ + + +Range stringifier + + + +
Test div
+
Another div
+
Last div
+
+