RESOLVED FIXED 147657
Replace JSFunctionNameScope with JSLexicalEnvironment for the function name scope. 
https://bugs.webkit.org/show_bug.cgi?id=147657
Summary Replace JSFunctionNameScope with JSLexicalEnvironment for the function name s...
Saam Barati
Reported 2015-08-04 15:35:00 PDT
We can just implement this as a "const"-like scope :)
Attachments
patch (61.17 KB, patch)
2015-08-04 19:05 PDT, Saam Barati 		mark.lam: review+
mark.lam: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Saam Barati
Comment 1 2015-08-04 19:05:51 PDT
Created attachment 258257 [details] patch
Mark Lam
Comment 2 2015-08-05 09:44:43 PDT
Comment on attachment 258257 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=258257&action=review r=me with fixes. Do we already have test coverage for all the aspects of the function name scope behavior? If not, please add the missing pieces. > Source/JavaScriptCore/ChangeLog:14 > + this variable like a "const" because You're cut off here mid thought.
Saam Barati
Comment 3 2015-08-05 11:16:44 PDT
(In reply to comment #2) > Comment on attachment 258257 [details] > patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=258257&action=review > > r=me with fixes. > > Do we already have test coverage for all the aspects of the function name > scope behavior? If not, please add the missing pieces. > We do have tests. I broke them and then fixed them :) > > Source/JavaScriptCore/ChangeLog:14 > > + this variable like a "const" because > > You're cut off here mid thought. Fixed.
Saam Barati
Comment 4 2015-08-05 11:30:20 PDT
landed in: http://trac.webkit.org/changeset/187969
Note You need to log in before you can comment on or make changes to this bug.