846316 - The instanceof operator raises exceptions when used in a sandbox with wantXrays=true

Status: RESOLVED WORKSFORME

(Firefox :: Untriaged, defect)

Description

[Anthony Lieuallen]

Attachment: instanceof-scratchpad-case.js

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:19.0) Gecko/20100101 Firefox/19.0
Build ID: 20130215130331

Steps to reproduce:

Get this upstream bug report:
https://github.com/greasemonkey/greasemonkey/issues/1716

Reproduce like so:

1. devtools.chrome.enabled = true
2. Open Scratchpad.
3. Environment > Browser
4. Run the attached script.


Actual results:

I got an "ok" alert from the wantXrays=false sandbox, then an "err" alert from the wantXrays=true sandbox, running the same code.


Expected results:

The instanceof operator should have worked regardless of xrays.

Comment 2

[Anthony Lieuallen]

This bug is not (completely) fixed.  While bug 822399 fixed this for "Event" other objects still break.  

Try the original test case with "Blob" substituted for "Event".  With Firefox nightly as of 50d25e083421 I see the same failure as from the original description.

Also fails: File.

Comment 3

[Peter Van der Beken [:peterv]]

Sure, those will be fixed too when they are converted to WebIDL.

Comment 4

[Liviu Cirdei]

Is this still an issue on latest Firefox versions? e.g. Firefox 43 or Nightly 46

Comment 5

[Anthony Lieuallen]

On most pages, no, resolved.

On Bugzilla pages (which seem to define a different object named Event), I get a different error, when wantXrays=false.  Which might be expected.

Comment 6

[Liviu Cirdei]

Thank you for your prompt response. If the initial issue is resolved, I will mark this as Resolved Worksforme.

Regarding the Bugzilla pages that trow a different error (with wantXrays=false), I think you should open a new bug report if you think the behavior is not the expected one or you could reopen this one if you consider it more useful.