Bug 14258 - Response time vs. download time should be shown in the network profile
Summary: Response time vs. download time should be shown in the network profile
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 523.x (Safari 3)
Hardware: All All
: P2 Normal
Assignee: David Smith
URL: http://boredzo.org/blog
Keywords: InRadar
: 18034 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-06-20 21:29 PDT by David Smith
Modified: 2008-04-14 09:26 PDT (History)
1 user (show)

See Also:


Attachments
Initial patch (4.45 KB, patch)
2008-04-13 01:47 PDT, David Smith
no flags Details | Formatted Diff | Diff
This tweaks opacities to look right, and sorts by response received time. (4.98 KB, patch)
2008-04-13 21:38 PDT, David Smith
timothy: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Smith 2007-06-20 21:29:42 PDT
A page I just loaded took most of its time downloading a 17k document, despite the ~300k of images. Clearly this wasn't a result of network issues, but instead server latency. Being able to see this on the graph (perhaps darkening the section of the graph in which data is actually loading) would make this much clearer for diagnosing pageload perf problems.

Note: the url may not demonstrate the issue well; it's just the one I saw it at
Comment 1 Adam Roben (:aroben) 2008-01-29 10:54:04 PST
<rdar://problem/5712780>
Comment 2 Adam Roben (:aroben) 2008-03-24 05:51:28 PDT
*** Bug 18034 has been marked as a duplicate of this bug. ***
Comment 3 David Smith 2008-04-13 01:47:33 PDT
Created attachment 20498 [details]
Initial patch

This only changes the display of the individual resource bars, not the bar for totals.
Comment 4 Timothy Hatcher 2008-04-13 11:36:47 PDT
Can you attach a screenshot?
Comment 5 David Smith 2008-04-13 12:42:47 PDT
(In reply to comment #4)
> Can you attach a screenshot?
> 

http://dscoder.com/inspectorwaitingtime.png
Comment 6 Timothy Hatcher 2008-04-13 20:32:08 PDT
I think we need to update the sorting function to sort by response received and end time.
Comment 7 David Smith 2008-04-13 21:34:19 PDT
(In reply to comment #6)
> I think we need to update the sorting function to sort by response received and
> end time.
> 

That ends up looking like this: http://dscoder.com/responsesorting.png

I'm not actually sure which one I prefer... the new one is a little odd, although I certainly see the logic of it.
Comment 8 David Smith 2008-04-13 21:38:31 PDT
Created attachment 20516 [details]
This tweaks opacities to look right, and sorts by response received time.
Comment 9 David Smith 2008-04-13 21:39:38 PDT
Argh, it also has a tab in it. Fixed locally.
Comment 10 David Smith 2008-04-14 09:26:37 PDT
Landed in r31863