577984 - Array.indexOf and XPCNativeWrapper could work together a little more cleanly

Status: RESOLVED INACTIVE

(Core :: XPConnect, defect)

Description

[Alex Vincent [:WeirdAl]]

Scenario:
var x = [elem, elem2, elem3, elem4] where all are DOM elements
x.indexOf(foo) == -1, where foo is a XPCNativeWrapper of elem4

This was causing a Mochitest of mine to fail.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Why would you expect that to work?  If you have:

  var x = [1, 2, 3];
  var y = x.indexOf("1");

would you expect anything other than -1?  The XPCNativeWrapper situation is no different....

Comment 2

[Alex Vincent [:WeirdAl]]

My opinion is that XPCNativeWrapper is a special case.  For instance, if I said (following up on comment 0):

var y = x[3];
(foo == y) resulted in true in my Mochitest.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Yes, and |1 == "1"| is also true.  I did pick my example pretty carefully.  What's special about XPCNativeWrapper here?

Comment 4

[BMO Automation]

Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.