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

@yoavweiss commented on this pull request.



>  </dl>
 
+<p>A <dfn>fetch timing info</dfn> is a <a for=/>struct</a> used to maintain timing information
+later required by the resource timing and navigation timiing specs. It has the following <a for=struct>items</a>:

s/timiing/timing/
Also It might be worthwhile to move the background info to a note

> +  <dt><dfn for="fetch timing info">response end time</dfn> (default zero)
+  <dt><dfn for="fetch timing info">alpn negotiated protocol</dfn> (default null)
+  <dd>Null or a string.
+
+  <dt><dfn for="fetch timing info">worker start time</dfn> (default zero)
+  <dt><dfn for="fetch timing info">domain lookup start time</dfn> (default zero)
+  <dt><dfn for="fetch timing info">domain lookup end time</dfn> (default zero)
+  <dt><dfn for="fetch timing info">connection start time</dfn> (default zero)
+  <dt><dfn for="fetch timing info">connection end time</dfn> (default zero)
+  <dt><dfn for="fetch timing info">secure connection start time</dfn> (default zero)
+  <dt><dfn for="fetch timing info">request start time</dfn> (default zero)
+  <dt><dfn for="fetch timing info">transfer size</dfn> (default zero)
+  <dt><dfn for="fetch timing info">encoded body size</dfn> (default zero)
+  <dt><dfn for="fetch timing info">decoded body size</dfn> (default zero)
+</dl>
+

Spurious new line?

> @@ -3348,6 +3396,8 @@ the request.
  <li><p>Run <a>main fetch</a> given <var>fetchParams</var>.
 </ol>
 
+<h3 id="sec-current-time">Current Time</h3>
+The <dfn>current time</dfn> is the present time given the <a for=/>global monotonic clock</a>.

Yeah, we definitely want this to point at https://w3c.github.io/hr-time/#dfn-current-high-resolution-time

-- 
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#pullrequestreview-602195864