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

@annevk commented on this pull request.



>     <li><p>Set <var>response</var> to the result of invoking <a for=/>handle fetch</a> for
    <var>requestForServiceWorker</var>. [[!HTML]] [[!SW]]
 
    <li>
     <p>If <var>response</var> is not null, then:
 
     <ol>
+     <li><p>Set <var>fetchParams</var>'s <a for="fetch params">timing info</a>'s
+     <a for="fetch timing info">response start time</a> to the <a for=/>unsafe shared current time</a>.
+
+     <li><p>Set <var>timingInfo</var>'s <a for="fetch timing info">encoded body size</a> and
+     <a for="fetch timing info">decoded body size</a> to <var>response</var>'s
+     `<code>Content-Length</code>` <a for=/>header</a>.

The problem is what it sets the field to. "response's Content-Length header" is not a thing. A response has a header list and from there you need to get the header and then you need to parse it into a value. Extract a length does that for `Content-Length`.

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