Summary: | Web Inspector: arrow function names are never inferred, call frames are labeled (anonymous function) | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | BJ Burg <bburg> | ||||||||||
Component: | Web Inspector | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | bburg, commit-queue, graouts, gskachkov, joepeck, mattbaker, nvasilyev, saam, timothy, webkit-bug-importer, ysuzuki | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Attachments: |
|
Created attachment 259640 [details]
Current call frames appearance in inspector.
As I understand this is related to the SetFunctionName operation described in ES6 specification http://www.ecma-international.org/ecma-262/6.0/#sec-setfunctionname So arrow function should support name property. let a = () => {}; a.name === 'a'; //TRUE Ohh I see that there is used anther property inferredName. Created attachment 261656 [details]
Patch
Tiny fix, but need a lot of time until I can check it in Web Inspector because of the long build
Created attachment 261673 [details]
Fixed_version
Looks like fixed
Comment on attachment 261656 [details]
Patch
r=me
I like one line bug fixes.
Comment on attachment 261656 [details] Patch Clearing flags on attachment: 261656 Committed r190066: <http://trac.webkit.org/changeset/190066> All reviewed patches have been landed. Closing bug. |
Created attachment 259638 [details] Test case In a simple case like let a = () => {} JSC should be able to infer this function's name as 'a'.