RESOLVED FIXED 149799
Web Inspector: Error displayed as an object when passed as a second argument to console.log
https://bugs.webkit.org/show_bug.cgi?id=149799
Summary Web Inspector: Error displayed as an object when passed as a second argument ...
Nikita Vasilyev
Reported 2015-10-04 20:19:36 PDT
Created attachment 262414 [details] [Image] Bug try { bar(); } catch(error) { console.log(error); console.log("prefix", error); } Expected: Both errors rendered using ErrorObjectView. Actual: console.log("prefix", error) shows an error as an object outline.
Attachments
[Image] Bug (98.66 KB, image/png)
2015-10-04 20:19 PDT, Nikita Vasilyev
no flags
WIP (226.64 KB, patch)
2015-10-04 21:16 PDT, Nikita Vasilyev
nvasilyev: commit-queue-
Patch (5.76 KB, patch)
2015-10-05 07:47 PDT, Nikita Vasilyev
no flags
[Animated GIF] With the patch applied (327.80 KB, image/gif)
2015-10-05 07:56 PDT, Nikita Vasilyev
no flags
Radar WebKit Bug Importer
Comment 1 2015-10-04 20:19:49 PDT
Nikita Vasilyev
Comment 2 2015-10-04 21:16:40 PDT
Nikita Vasilyev
Comment 3 2015-10-05 07:47:18 PDT
Nikita Vasilyev
Comment 4 2015-10-05 07:56:09 PDT
Created attachment 262434 [details] [Animated GIF] With the patch applied
Joseph Pecoraro
Comment 5 2015-10-05 12:15:41 PDT
Comment on attachment 262433 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=262433&action=review r=me > Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js:506 > + const fragment = document.createDocumentFragment(); Maybe its just me, but I think `let` is better than `const` in some of these cases.
WebKit Commit Bot
Comment 6 2015-10-05 13:00:54 PDT
Comment on attachment 262433 [details] Patch Clearing flags on attachment: 262433 Committed r190571: <http://trac.webkit.org/changeset/190571>
WebKit Commit Bot
Comment 7 2015-10-05 13:01:01 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.