Closed Bug 1309239 Opened 8 years ago Closed 8 years ago

Don't check for Xray slot bits for interfaces that never have Xrays

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla52
Tracking Flags:
Tracking Status
firefox52 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(1 file)

Fix the codegen for a cached DOM getter to not worry about Xrays if the interface doesn't want Xrays (i.e. is not mainthread-exposed)
3.54 KB, patch
baku
: review+
Details | Diff | Splinter Review 
Andrea noticed this in bug 1308956.  We don't include XrayWrapper.h if the interface doesn't want Xrays, but the code we spit out in the getter still checks for them.  Simple fix coming up.
Attachment #8799760 - Flags: review?(amarchesini) → review+
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7eeb973656fa
Fix the codegen for a cached DOM getter to not worry about Xrays if the interface doesn't want Xrays (i.e. is not mainthread-exposed).  r=baku
https://hg.mozilla.org/mozilla-central/rev/7eeb973656fa
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: