wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Backing out changeset 93104:1b954a5da88c (fix for bug 489623) since its still crashing on AWFY

Browse Source
This commit is contained in:
Kannan Vijayan 2012-04-30 16:07:05 -04:00
parent 77809391d1
commit 395e8b4add
1 changed files with 1 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
js/src/ion/IonCaches.cpp
View File

@ -456,10 +456,6 @@ IsEligibleForInlinePropertyAdd(JSContext *cx, JSObject *obj, jsid propId, uint32
if (!propShape || propShape->inDictionary() || !propShape->hasSlot() || !propShape->hasDefaultSetter())
return false;
// If object has a non-default resolve hook, don't inline
if (obj->getClass()->resolve != JS_ResolveStub)
return false;
// walk up the object prototype chain and ensure that all prototypes
// are native, and that all prototypes have no getter or setter
// defined on the property
@ -470,21 +466,7 @@ IsEligibleForInlinePropertyAdd(JSContext *cx, JSObject *obj, jsid propId, uint32
// if prototype defines this property in a non-plain way, don't optimize
const Shape *protoShape = proto->nativeLookup(cx, propId);
if (protoShape) {
// If the prototype has a property with this name, we may be able to
// inline-add if the property has a default setter. If there's no
// default setter, then setting would potentially cause special
// behaviour, so we choose not to optimize.
if (protoShape->hasDefaultSetter()) {
*propShapeOut = propShape;
return true;
}
return false;
}
// Otherise, if there's no such property, watch out for a resolve hook that would need
// to be invoked and thus prevent inlining of property addition.
if (proto->getClass()->resolve != JS_ResolveStub)
if (protoShape && !protoShape->hasDefaultSetter())
return false;
}