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

Bug 821850 - Properly propagate |strict| in JSObject::deleteByValue. r=jorendorff

Browse Source
This commit is contained in:
Bobby Holley 2013-02-08 14:24:19 +00:00
parent 340661f4dc
commit bae9282c5c
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
js/src/jit-test/tests/basic/bug821850.js Normal file
View File

@ -0,0 +1,9 @@
load(libdir + "asserts.js");
m={}
Object.defineProperty(m, 'p', {value: 3});
assertThrowsInstanceOf(function() {"use strict"; delete m.p;}, TypeError);
x = new Proxy(m, {})
assertEq(x.p, 3);
assertEq((function fun() {"use strict"; return delete x.p; })(), false);

4
js/src/jsobj.cpp
View File

@ -1623,10 +1623,10 @@ JSObject::deleteByValue(JSContext *cx, HandleObject obj,
return false;
if (name->isIndex(&index))
return deleteElement(cx, obj, index, rval, false);
return deleteElement(cx, obj, index, rval, strict);
Rooted<PropertyName*> propname(cx, name->asPropertyName());
return deleteProperty(cx, obj, propname, rval, false);
return deleteProperty(cx, obj, propname, rval, strict);
}
JS_FRIEND_API(bool)