Bug 14258

Summary:

Response time vs. download time should be shown in the network profile

Product:

WebKit

Reporter:

David Smith <catfish.man>

Component:

Web Inspector (Deprecated)

Assignee:

David Smith <catfish.man>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

timothy

Priority:

Keywords:

InRadar

Version:

523.x (Safari 3)

Hardware:

All

OS:

All

URL:

http://boredzo.org/blog

Attachments:

Description	Flags
Initial patch	none
This tweaks opacities to look right, and sorts by response received time.	timothy: review+

David Smith

Reported 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

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
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Adam Roben (:aroben)

Comment 1 2008-01-29 10:54:04 PST

<rdar://problem/5712780>

Adam Roben (:aroben)

Comment 2 2008-03-24 05:51:28 PDT

*** Bug 18034 has been marked as a duplicate of this bug. ***

David Smith

Comment 3 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.

Timothy Hatcher

Comment 4 2008-04-13 11:36:47 PDT

Can you attach a screenshot?

David Smith

Comment 5 2008-04-13 12:42:47 PDT

(In reply to comment #4) > Can you attach a screenshot? > http://dscoder.com/inspectorwaitingtime.png

Timothy Hatcher

Comment 6 2008-04-13 20:32:08 PDT

I think we need to update the sorting function to sort by response received and end time.

David Smith

Comment 7 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.

David Smith

Comment 8 2008-04-13 21:38:31 PDT

Created attachment 20516 [details] This tweaks opacities to look right, and sorts by response received time.

David Smith

Comment 9 2008-04-13 21:39:38 PDT

Argh, it also has a tab in it. Fixed locally.

David Smith

Comment 10 2008-04-14 09:26:37 PDT

Landed in r31863

Note You need to log in before you can comment on or make changes to this bug.