Open
Bug 1137739
Opened 10 years ago
Updated 2 years ago
Remove the nsIXPConnectWrappedNative bits from xpc::NativeGlobal
Categories
(Core :: XPConnect, defect)
Tracking
()
NEW
People
(Reporter: bzbarsky, Unassigned)
References
(Depends on 2 open bugs, Blocks 1 open bug)
Details
Once bug 888600 is fixed and all our globals are non-XPCWN, we can do this.
Reporter | ||
Comment 1•10 years ago
|
||
I guess we'd need to also convert BackstagePass, though, right?
Comment 2•10 years ago
|
||
(In reply to Not doing reviews right now from comment #1)
> I guess we'd need to also convert BackstagePass, though, right?
Yes, exactly. I came to the bug to comment as such. :-)
Reporter | ||
Updated•10 years ago
|
Comment 3•7 years ago
|
||
Fixing this would help fix bug 1274786, which shows up a tiny bit in speedometer.
Reporter | ||
Comment 4•7 years ago
|
||
OK, so we need to fix bug 888600 as a start... ;)
Comment 5•7 years ago
|
||
right. Or do we have some spare bits somewhere we could use to indicate whether the object is nsIGlobalObject and if so, just do static_cast?
Reporter | ||
Comment 6•7 years ago
|
||
The object is certainly an nsIGlobalObject. The question is what the offset of its nsIGlobalObject pointer is from its canonical nsISupports pointer.
That is, this bug is about the !native case, which has a QI to nsIXPConnectWrappedNative and whatnot. But that codepath is not taken for Window at this point; it only happens for non-webidl globals like the ones in bug 888600, sandboxes, and backstagepass.
If you're seeing QIing on speedometer, that's probably coming from the QI to nsIGlobalObject which isn't really related to this bug...
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•