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

No. It sets `startTime` to be the `redirect start time`, so it exposes a `startTime`, and the caller doesn't know that its from a 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_r600685613

Received on Wednesday, 24 March 2021 17:06:18 UTC