119325 2013-07-30 23:37:09 -0700 compareDocumentPosition() reports an attribute and its content as disconnected 2024-03-18 10:15:52 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) Unspecified Unspecified RESOLVED CONFIGURATION CHANGED P2 Normal --- 1 cdumez webkit-unassigned ahmad.saleem792 annevk ap darin rniwa oldest_to_newest 913205 0 cdumez 2013-07-30 23:37:09 -0700 Node::compareDocumentPosition() reports an attribute and its content as disconnected. Instead, the attribute should contain and precede its content. This is causing the following test case to fail: LayoutTests/dom/xhtml/level3/core/nodecomparedocumentposition38.xhtml I investigated the issue a bit and it seems the problem seems to be that Node::inDocument() returns false for the Text node (content) instead of true. Note that there is already a workaround in place in Node::compareDocumentPositionInternal() that addresses the same issue for Attribute nodes. See comment in that method: // Note that we avoid comparing Attr nodes here, since they return false from inDocument() all the time (which seems like a bug). It is likely we need to extend the workaround to Text nodes or fix the underlying problem (have inDocument() return correct values for Attr and Text nodes). Specification: http://dom.spec.whatwg.org/#dom-node-comparedocumentposition Corresponding Blink bug: http://code.google.com/p/chromium/issues/detail?id=264138 913207 1 cdumez 2013-07-30 23:39:16 -0700 As explained by Adam in http://code.google.com/p/chromium/issues/detail?id=264138#c1, Attr should no longer be a Node according to the latest specification. It should be its own interface [1]. If we followed the spec, the bug would disappear. [1] http://dom.spec.whatwg.org/#attr 2021668 2 ahmad.saleem792 2024-03-17 15:30:35 -0700 Safari is passing all WPT tests: https://wpt.fyi/results/dom/nodes/Node-compareDocumentPosition.html?label=master&label=experimental&aligned= Chris - Do we need to anything here? 2021797 3 rniwa 2024-03-18 10:15:52 -0700 Looks like this bug has been fixed by now.