<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Frontend should have access to Resource Timing information"
href="https://bugs.webkit.org/show_bug.cgi?id=160095#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Frontend should have access to Resource Timing information"
href="https://bugs.webkit.org/show_bug.cgi?id=160095">bug 160095</a>
from <span class="vcard"><a class="email" href="mailto:achristensen@apple.com" title="Alex Christensen <achristensen@apple.com>"> <span class="fn">Alex Christensen</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=284369&action=diff" name="attach_284369" title="WIP Patch - Timing info">attachment 284369</a> <a href="attachment.cgi?id=284369&action=edit" title="WIP Patch - Timing info">[details]</a></span>
WIP Patch - Timing info
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=284369&action=review">https://bugs.webkit.org/attachment.cgi?id=284369&action=review</a>
Instead of doing this, I think we should finish implementing the resource timing api and use that data in the inspector instead of collecting data just for the inspector. This would also require collecting new timing points, so it would be conceptually similar.
<span class="quote">> Source/WebCore/inspector/InspectorNetworkAgent.cpp:187
> +Ref<Inspector::Protocol::Network::ResourceTiming> InspectorNetworkAgent::buildObjectForTiming(const ResourceLoadTiming& timing, ResourceLoader* resourceLoader)</span >
This should be a ResourceLoader&
<span class="quote">> Source/WebCore/platform/network/ResourceLoadTiming.h:98
> - int secureConnectionStart;
> + double domainLookupStart;</span >
This is an unrelated change that should be in a different patch. It changes the resolution of the navigation timing api.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>