Bug 180998 – Web Inspector: Network Table - Update the Time column to include the total duration not just the download duration

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

Network Table - Update the Time column to include the total duration not just the download duration

It was confusing to have the "Time" not match the popover. Previously it was just the download duration, which excluded the queueing / connection time, which can grow large. Just make this the total time.

[Radar WebKit Bug Importer]

<rdar://problem/36140424>

[Joseph Pecoraro]

Created attachment 329822 [details]
[PATCH] Proposed Fix

[Matt Baker]

Comment on attachment 329822 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=329822&action=review

r=me, with a comment.

> Source/WebInspectorUI/UserInterface/Models/Resource.js:545
> +    get downloadDuration()

Are there plans to use `downloadDuration` in the very near future? If not let's remove it. I think `totalDuration` makes sense as a name event with out, since we have a `receiveDuration`.

[Joseph Pecoraro]

(In reply to Matt Baker from comment #3)
> Comment on attachment 329822 [details]
> [PATCH] Proposed Fix
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=329822&action=review
> 
> r=me, with a comment.
> 
> > Source/WebInspectorUI/UserInterface/Models/Resource.js:545
> > +    get downloadDuration()
> 
> Are there plans to use `downloadDuration` in the very near future? If not
> let's remove it. I think `totalDuration` makes sense as a name event with
> out, since we have a `receiveDuration`.

Okay, I'll remove it.

[Joseph Pecoraro]

<https://trac.webkit.org/r226151>