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 Wednesday, October 28, 2009 11:23:38 PM UTC
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 Wednesday, October 28, 2009 11:26:11 PM UTC
Created attachment 42059 [details] [PATCH] Glue
Pavel Feldman
Comment 2 Wednesday, October 28, 2009 11:28:19 PM UTC
Created attachment 42060 [details] [IMAGE] See how nicely 2127 DOM Events are glued. Saved so much space!
Timothy Hatcher
Comment 3 Wednesday, October 28, 2009 11:58:16 PM UTC
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 Thursday, October 29, 2009 12:19:24 AM UTC
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.