RESOLVED FIXED 42620
Web Inspector: it is possible to show call-stack instead of caller property for Timeline records
https://bugs.webkit.org/show_bug.cgi?id=42620
Summary Web Inspector: it is possible to show call-stack instead of caller property f...
Ilya Tikhonovsky
Reported 2010-07-20 01:18:04 PDT
%subj%
Attachments
[patch] initial version. (6.18 KB, patch)
2010-07-20 03:02 PDT, Ilya Tikhonovsky
no flags
screenshot (32.07 KB, image/png)
2010-07-20 03:09 PDT, Ilya Tikhonovsky
no flags
[patch] second iteration (6.70 KB, patch)
2010-07-20 06:46 PDT, Ilya Tikhonovsky
no flags
screenshot (34.65 KB, image/png)
2010-07-20 06:47 PDT, Ilya Tikhonovsky
no flags
[patch] third iteration. (8.48 KB, patch)
2010-07-20 08:42 PDT, Ilya Tikhonovsky
pfeldman: review+
screenshot (43.29 KB, image/png)
2010-07-20 08:42 PDT, Ilya Tikhonovsky
no flags
Ilya Tikhonovsky
Comment 1 2010-07-20 03:02:27 PDT
Created attachment 62048 [details] [patch] initial version.
Ilya Tikhonovsky
Comment 2 2010-07-20 03:09:21 PDT
Created attachment 62051 [details] screenshot
Yury Semikhatsky
Comment 3 2010-07-20 03:17:17 PDT
(In reply to comment #2) > Created an attachment (id=62051) [details] > screenshot The tooltip seems to be overloaded with stack data. We could try to make timeline records expandable and show the stack trace in it.
Ilya Tikhonovsky
Comment 4 2010-07-20 06:46:32 PDT
Created attachment 62069 [details] [patch] second iteration
Ilya Tikhonovsky
Comment 5 2010-07-20 06:47:00 PDT
Created attachment 62070 [details] screenshot
Pavel Feldman
Comment 6 2010-07-20 07:25:59 PDT
Should we align the frames by "@"?
Ilya Tikhonovsky
Comment 7 2010-07-20 08:42:28 PDT
Created attachment 62081 [details] [patch] third iteration.
Ilya Tikhonovsky
Comment 8 2010-07-20 08:42:52 PDT
Created attachment 62082 [details] screenshot
Ilya Tikhonovsky
Comment 9 2010-07-20 12:16:14 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/English.lproj/localizedStrings.js M WebCore/inspector/front-end/TimelinePanel.js M WebCore/inspector/front-end/inspector.css Committed r63757
Note You need to log in before you can comment on or make changes to this bug.