[webkit-reviews] review requested: [Bug 7831] .apply/call does not
set this for eval : [Attachment 7146] Modified Patch
bugzilla-request-daemon at opendarwin.org
bugzilla-request-daemon at opendarwin.org
Sat Mar 18 01:25:18 PST 2006
Francisco Tolmasky <tolmasky at gmail.com> has asked for review:
Bug 7831: .apply/call does not set this for eval
http://bugzilla.opendarwin.org/show_bug.cgi?id=7831
Attachment 7146: Modified Patch
http://bugzilla.opendarwin.org/attachment.cgi?id=7146&action=edit
------- Additional Comments from Francisco Tolmasky <tolmasky at gmail.com>
I ended up having to make a significantly different patch for this. The old
one, while working, caused a regression in Expressions/1.11.1.js or so. The
problem is that callAsFunction always gets sent a thisObj, so its not enough to
check whether thisObj isn't null or undefined to see whether we are actually
being called from an apply or call. So the solution I came up with was to
trick it by creating an inbetween ExecState that contains the correct thisObj.
I wanted to float this by you guys as whether it was an acceptible solution or
not before writing a changelog/tests for it, since it does seem a bit hackish.
Also, this new patch fixes js1_6/Array/regress-304828.js .
More information about the webkit-reviews
mailing list