Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-07-23 15:23:42 +01:00
Code Issues Projects Releases Packages Wiki Activity

Auto merge of #6559 - kball:IS1824-implement-Node.baseURI, r=Ms2ger

Browse source 
Implement Node.baseURI

Addresses Issue #1824.  Implements Node.baseURI based on https://www.w3.org/Bugs/Public/show_bug.cgi?id=20976#c32

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6559)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2015-07-06 11:04:45 -06:00
commit 1e16047846
5 changed files with 42 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

7
components/script/dom/node.rs
View file

@ -987,7 +987,7 @@ impl<'a> NodeHelpers for &'a Node {
fn summarize(self) -> NodeInfo {
NodeInfo {
uniqueId: self.get_unique_id(),
baseURI: self.GetBaseURI().unwrap_or("".to_owned()),
baseURI: self.BaseURI(),
parent: self.GetParentNode().map(|node| node.r().get_unique_id()).unwrap_or("".to_owned()),
nodeType: self.NodeType(),
namespaceURI: "".to_owned(), //FIXME
@ -1950,9 +1950,8 @@ impl<'a> NodeMethods for &'a Node {
}
// https://dom.spec.whatwg.org/#dom-node-baseuri
fn GetBaseURI(self) -> Option<DOMString> {
// FIXME (#1824) implement.
None
fn BaseURI(self) -> DOMString {
self.owner_doc().URL()
}
// https://dom.spec.whatwg.org/#dom-node-ownerdocument

2
components/script/dom/webidls/Node.webidl
View file

@ -26,7 +26,7 @@ interface Node : EventTarget {
readonly attribute DOMString nodeName;
[Pure]
readonly attribute DOMString? baseURI;
readonly attribute DOMString baseURI;
[Pure]
readonly attribute Document? ownerDocument;

4
tests/wpt/metadata/MANIFEST.json
View file

@ -13117,6 +13117,10 @@
"path": "dom/nodes/Node-appendChild.html",
"url": "/dom/nodes/Node-appendChild.html"
},
{
"path": "dom/nodes/Node-baseURI.html",
"url": "/dom/nodes/Node-baseURI.html"
},
{
"path": "dom/nodes/Node-childNodes.html",
"url": "/dom/nodes/Node-childNodes.html"

2
tests/wpt/metadata/html/infrastructure/urls/terminology-0/document-base-url.html.ini
View file

@ -5,7 +5,7 @@
expected: FAIL
[The fallback base URL of a document containing no base element is the document's address.]
expected: FAIL
expected: PASS
[The fallback base URL of a document whose address is about:blank is the document base URL of the creator document.]
expected: TIMEOUT

32
tests/wpt/web-platform-tests/dom/nodes/Node-baseURI.html Normal file
View file

@ -0,0 +1,32 @@
<!DOCTYPE html>
<title>Node.baseURI</title>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<div id="log"></div>
<script>
test(function() {
var element = document.createElement("div");
document.body.appendChild(element);
assert_equals(element.baseURI, document.URL);
}, "For elements belonging to document, baseURI should be document url")
test(function() {
var element = document.createElement("div");
assert_equals(element.baseURI, document.URL);
}, "For elements unassigned to document, baseURI should be document url")
test(function() {
var fragment = document.createDocumentFragment();
var element = document.createElement("div");
fragment.appendChild(element);
assert_equals(element.baseURI, document.URL)
}, "For elements belonging to document fragments, baseURI should be document url")
test(function() {
var fragment = document.createDocumentFragment();
var element = document.createElement("div");
fragment.appendChild(element);
document.body.appendChild(fragment);
assert_equals(element.baseURI, document.URL)
}, "After inserting fragment into document, element baseURI should be document url")
</script>