RESOLVED FIXED 30885
Web Inspector: Glue subsequent timeline records with same category and title together.
https://bugs.webkit.org/show_bug.cgi?id=30885
Summary Web Inspector: Glue subsequent timeline records with same category and title ...
Pavel Feldman
Reported 2009-10-28 15:23:38 PDT
I chose 100ms (which is a bit high) since it was gluing mouse events together nicely. But given that there is a counter and that the constraints on gluing are rather strict, I think it is Ok.
Attachments
[PATCH] Glue (6.31 KB, patch)
2009-10-28 15:26 PDT, Pavel Feldman
timothy: review+
[IMAGE] See how nicely 2127 DOM Events are glued. Saved so much space! (206.22 KB, image/png)
2009-10-28 15:28 PDT, Pavel Feldman
no flags
Pavel Feldman
Comment 1 2009-10-28 15:26:11 PDT
Created attachment 42059 [details] [PATCH] Glue
Pavel Feldman
Comment 2 2009-10-28 15:28:19 PDT
Created attachment 42060 [details] [IMAGE] See how nicely 2127 DOM Events are glued. Saved so much space!
Timothy Hatcher
Comment 3 2009-10-28 15:58:16 PDT
Comment on attachment 42059 [details] [PATCH] Glue Don't land the WebInspector.UIString change. I am fine enabling the Panel if you are.
Pavel Feldman
Comment 4 2009-10-28 16:19:24 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.js Committed r50246
Note You need to log in before you can comment on or make changes to this bug.