RESOLVED FIXED Bug 16085
Web Inspector's Network Timeline graph code clean up and misc. fixes 
https://bugs.webkit.org/show_bug.cgi?id=16085
Summary Web Inspector's Network Timeline graph code clean up and misc. fixes
Timothy Hatcher
Reported 2007-11-21 12:12:19 PST
The Network Timeline graph code has a couple of problems. - It defines a few global functions that should be nested or hanging off another object/prototype. - It recomputes the segment percentages twice, once for the main pill and another time for the reflection. - Sometimes the percentages will round down so the total is below 100%. (We already account for a total greater than 100%.) - The pill shadow is drawn with a stoke on a pixel boundary, to achieve a blurry line. We should use the canvas shadow drawing properties.
Attachments
Patch (15.31 KB, patch)
2007-11-21 12:18 PST, Timothy Hatcher 		aroben: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Timothy Hatcher
Comment 1 2007-11-21 12:18:50 PST
Created attachment 17434 [details] Patch
Timothy Hatcher
Comment 2 2007-11-21 12:20:02 PST
This patch also removes un-needed save/restore calls.
Adam Roben (:aroben)
Comment 3 2007-11-21 15:04:55 PST
Comment on attachment 17434 [details] Patch r=me
Timothy Hatcher
Comment 4 2007-11-21 15:40:45 PST
Landed in r27943. http://trac.webkit.org/projects/webkit/changeset/27943
Note You need to log in before you can comment on or make changes to this bug.