Bug 42620

Summary: Web Inspector: it is possible to show call-stack instead of caller property for Timeline records
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Ilya Tikhonovsky <loislo>
Status: RESOLVED FIXED    
Severity: Normal CC: bweinstein, joepeck, keishi, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[patch] initial version.
none
screenshot
none
[patch] second iteration
none
screenshot
none
[patch] third iteration.
pfeldman: review+
screenshot none

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
DetailsFormatted DiffDiff
screenshot (32.07 KB, image/png)
2010-07-20 03:09 PDT, Ilya Tikhonovsky 		no flags
Details
[patch] second iteration (6.70 KB, patch)
2010-07-20 06:46 PDT, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
screenshot (34.65 KB, image/png)
2010-07-20 06:47 PDT, Ilya Tikhonovsky 		no flags
Details
[patch] third iteration. (8.48 KB, patch)
2010-07-20 08:42 PDT, Ilya Tikhonovsky 		pfeldman: review+
DetailsFormatted DiffDiff
screenshot (43.29 KB, image/png)
2010-07-20 08:42 PDT, Ilya Tikhonovsky 		no flags
Details
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
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.