47884 - compareNode return value is wrong in Range.idl & impls

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

[Heikki Toivonen (remove -bugzilla when emailing directly)]

The compareNode return value is wrong in Range.idl. Or actually, right now the
IDL file is correct, but the implementation(s) return value is different. It
compiles because DOM IDL is not part of normal build. Once me or Saari checks in
joki's patch, the Range.idl will also contain the wrong return value. This bug
is here to make sure that we will eventually fix the return value in all cases.

Is there a bug for enabling DOM IDL build by default? If we could enable it by
default on at least some platforms we would have noticed this kind of thing
earlier...

I do not think compareNode is part of any DOM level spec, but I put this into
DOM 2 component.

Comment 1

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Architecture issue. There are quite a few places that need to be changed, but it
is a simple thing, changing PRInt16 to PRUint16 (and making sure we don't break
anything by doing this).

Nominating for nsbeta3.

Comment 2

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Adding Chris to CC.

Comment 3

[Nisheeth Ranjan]

Marking nsbeta3+

Comment 4

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Seems like Chris fixed this already in the big landing. Marking fixed.