Closed Bug 1451017 Opened 4 years ago Closed 3 years ago

Remove Xray resolveNativeProperty

Categories

(Core :: XPConnect, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: evilpie, Assigned: evilpie)

References

Details

Attachments

(1 file)

Only XPCWN Xrays have an implementation of resolveNativeProperty that does anything.
Priority: -- → P3
This can't land without bug 1053271, so the review isn't urgent. With the removal of XPCWN Xrays all of the resolveNativeProperty implementations either crash or do nothing. This is a small incremental step before we can remove XrayWrapper<Base, Traits>::getPropertyDescriptor.
Attachment #8974148 - Flags: review?(bzbarsky)
Attachment #8974148 - Flags: review?(bzbarsky) → review+
Blocks: 1160757
Pushed by evilpies@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0d24499ad4e8
Remove Xray resolveNativeProperty. r=bz
https://hg.mozilla.org/mozilla-central/rev/0d24499ad4e8
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.