Open
Bug 1163746
Opened 9 years ago
Updated 2 years ago
Try removing JS_AlreadyHasOwnProperty
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
NEW
People
(Reporter: evilpie, Unassigned)
References
Details
bug 1163423 comment 2 > So looking at consumers of JS_AlreadyHasOwnProperty(ById): > > 1) DOMProxyShadows is using it on the expando, so could use > JS_HasOwnPropertyById. Assuming that's not slower, of course. > > 2) In DefineConstructor, I'm not exactly sure why we need that check. It > might be possible to remove it, if we know that the global's resolve hook > will always resolve the relevant properties before someone has a chance to > redefine them. > > 3) Plugin code, no idea. But those objects _do_ have resolve hooks, so > need to check carefully. > > 4) XBL, pretty sure is working on objects with no resolve hooks only. > > 5) Xrays, working with holder objects, which have no resolve hook.
Reporter | ||
Updated•9 years ago
|
Assignee: nobody → evilpies
Reporter | ||
Comment 1•9 years ago
|
||
I don't necessarily know more than bz here, so somebody else should take this.
Assignee: evilpies → nobody
Flags: needinfo?(bzbarsky)
Comment 2•9 years ago
|
||
I'm not going to have time for this in the near term. The hard part here is the plugin bits, of course.
Flags: needinfo?(bzbarsky)
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•