Remove some obsolete code in XrayWrapper.cpp

RESOLVED FIXED in Firefox 47

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: peterv, Assigned: peterv)

Tracking

unspecified
mozilla47
Points:
---

Firefox Tracking Flags

(firefox47 fixed)

Details

(Whiteboard: dom-triaged btpp-active)

Attachments

(1 attachment)

Posted patch v1Splinter Review
We haven't supported non-WebIDL Window for a while (bug 1017424), but there's some code left for XPCWN Xrays for Window.
Assignee

Updated

3 years ago
Attachment #8719593 - Flags: review?(bobbyholley)
Comment on attachment 8719593 [details] [diff] [review]
v1

Review of attachment 8719593 [details] [diff] [review]:
-----------------------------------------------------------------

\o/

I think we can probably actually just get rid of all XPCWN Xrays, given that it seems unlikely that any of the remaining XPCWN objects are used over Xray. If you have a moment, would you mind doing a try push to see what happens if we turn off XPCWN Xrays entirely?

r=me either way
Attachment #8719593 - Flags: review?(bobbyholley) → review+
Assignee: nobody → peterv
Whiteboard: dom-triaged btpp-active

Comment 3

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/cff999f3aee1
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.