Re: [whatwg/fetch] Initial resource timing integration. (#1185)

@noamr commented on this pull request.



> +<ol>
+ <li><p>If <var>response</var>'s <a for=response>URL list</a> is null or
+ <a for=list lt="is empty">empty</a>, then return.
+
+ <li><p>Let <var>originalURL</var> be <var>response</var>'s <a for=response>URL list</a>[0].
+
+ <li><p>Let <var>timingInfo</var> be <var>response</var>'s <a for=response>timing info</a>.
+
+ <li><p>If <var>timingInfo</var> is null, then return.
+
+ <li><p>Let <var>startTime</var> be <var>timingInfo</var>'s
+ <a for="fetch timing info">fetch start time</a>.
+
+ <li><p>If <var>timingInfo</var>'s <a for="fetch timing info">redirect start time</a> is not 0, then
+ set <var>startTime</var> to <var>timingInfo</var>'s
+ <a for="fetch timing info">redirect start time</a>.

https://github.com/w3c/resource-timing/issues/260 :)
This PR actually fixes that, as `startTime` would mean `redirectStart`, the time before the first redirect, and not `fetchStart`, the time after the last redirect.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1185#discussion_r600696899