Last Comment Bug 685139 - node.contains(node) returns false per spec and in Gecko, true in other browsers
: node.contains(node) returns false per spec and in Gecko, true in other browsers
: dev-doc-complete
Product: Core
Classification: Components
Component: DOM (show other bugs)
: Trunk
: All All
: -- normal (vote)
: ---
Assigned To: Olli Pettay [:smaug]
: Andrew Overholt [:overholt]
Depends on:
Blocks: 685086
  Show dependency treegraph
Reported: 2011-09-07 06:34 PDT by bugzilla33
Modified: 2012-01-30 05:26 PST (History)
6 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

contains.html (658 bytes, text/html)
2011-09-07 06:34 PDT, bugzilla33
no flags Details
patch + tests (3.40 KB, patch)
2011-09-07 07:14 PDT, Olli Pettay [:smaug]
bzbarsky: review+
Details | Diff | Splinter Review

Description bugzilla33 2011-09-07 06:34:29 PDT
Created attachment 558787 [details]

User Agent: Mozilla/5.0 (Windows NT 6.1; rv:9.0a1) Gecko/20110907 Firefox/9.0a1
Build ID: 20110907030853

Steps to reproduce:

1. open testcase
2. click TEST button

Actual results:

theSameElem.contains(theSameElem) returns FALSE

Expected results:

theSameElem.contains(theSameElem) should returns TRUE

like: IE9, Chrome 13, Safari 5.1, Opera 11.5, Konqueror 4.5
Comment 1 Mounir Lamouri (:mounir) 2011-09-07 06:40:28 PDT
This is not what the specs require but maybe they should change if all other browsers does it this way?
Comment 2 Boris Zbarsky [:bz] (still a bit busy) 2011-09-07 06:42:19 PDT
Requesting tracking to make sure we don't ship something incompatible here if the spec changes in the next few weeks...
Comment 3 Olli Pettay [:smaug] 2011-09-07 07:14:52 PDT
Created attachment 558794 [details] [diff] [review]
patch + tests
Comment 4 Boris Zbarsky [:bz] (still a bit busy) 2011-09-07 07:59:53 PDT
Comment on attachment 558794 [details] [diff] [review]
patch + tests

Comment 5 Olli Pettay [:smaug] 2011-09-07 13:18:58 PDT
Comment 6 Eric Shepherd [:sheppy] 2011-10-14 10:10:19 PDT
This is documented as implemented, but I've added a note that this is contrary to the specification.
Comment 7 Boris Zbarsky [:bz] (still a bit busy) 2011-10-14 10:17:27 PDT
Spec was changed to match implementations a while back.  I've removed the note from the devmo docs.
Comment 8 Eric Shepherd [:sheppy] 2011-10-14 10:19:23 PDT
Thanks bz. :)
Comment 9 Johnny Stenback (:jst, 2011-12-01 14:47:46 PST
This made 9, not tracking at this point.

Note You need to log in before you can comment on or make changes to this bug.