Summary: | Web Inspector: Stack trace view doesn't properly display lines without function names | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nikita Vasilyev <nvasilyev> | ||||||||
Component: | Web Inspector | Assignee: | Nikita Vasilyev <nvasilyev> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Local Build | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Description
Nikita Vasilyev
2015-10-08 04:18:44 PDT
Created attachment 262686 [details]
[HTML] Reduction
Created attachment 262687 [details]
Patch
Comment on attachment 262687 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=262687&action=review > Source/WebInspectorUI/UserInterface/Models/StackTrace.js:65 > + ({url, lineNumber, columnNumber} = WebInspector.StackTrace._parseLocation(line.slice(atIndex + 1))); Is the wrapping () needed here? > Source/WebInspectorUI/UserInterface/Models/StackTrace.js:67 > + ({url, lineNumber, columnNumber} = WebInspector.StackTrace._parseLocation(line)); Ditto. Comment on attachment 262687 [details]
Patch
It looks like this can be tested. Can we write a test for it?
(In reply to comment #4) > Comment on attachment 262687 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=262687&action=review > > > Source/WebInspectorUI/UserInterface/Models/StackTrace.js:65 > > + ({url, lineNumber, columnNumber} = WebInspector.StackTrace._parseLocation(line.slice(atIndex + 1))); > > Is the wrapping () needed here? > > > Source/WebInspectorUI/UserInterface/Models/StackTrace.js:67 > > + ({url, lineNumber, columnNumber} = WebInspector.StackTrace._parseLocation(line)); > > Ditto. Yes! I was surprised to discover this too. var x = 0; {x} = {x: 2}; // SyntaxError: Unexpected token '=' console.info(x); This, however, works as you'd expect: var {x} = {x: 2}; console.info(x); // 2 Created attachment 262747 [details]
Patch (with a test)
Comment on attachment 262747 [details] Patch (with a test) Clearing flags on attachment: 262747 Committed r190780: <http://trac.webkit.org/changeset/190780> All reviewed patches have been landed. Closing bug. |