[Webkit-unassigned] [Bug 145132] [ES6] Arrow function syntax. Arrow function specific features. Lexical bind "arguments"

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Jan 24 09:14:36 PST 2016


https://bugs.webkit.org/show_bug.cgi?id=145132

--- Comment #58 from GSkachkov <gskachkov at gmail.com> ---
Comment on attachment 269298
  --> https://bugs.webkit.org/attachment.cgi?id=269298
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=269298&action=review

Also I've added test of Generator

>> Source/JavaScriptCore/tests/stress/arrowfunction-lexical-bind-arguments-strict.js:32
>> +    // https://bugs.webkit.org/show_bug.cgi?id=152570
> 
> why does it lead to a reference error?

Now I understand that it is wrong place for this test. I've removed it. The idea is that Global scope does not contain variable 'arguments', and access to the 'arguments' from global scope should lead to ReferenceError. Current test is executed in JSC app where arguments is empty array.

>> Source/WebInspectorUI/UserInterface/Base/Object.js:63
>> +            that.removeEventListener(eventType, wrappedCallback, null);
> 
> why is this necessary?

See line below. Without this change, but with current patch - arguments will start contain values of singleFireEventListener instead of values that is passed during wrappedCallback invocation.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160124/aff3b082/attachment.html>


More information about the webkit-unassigned mailing list