Re: [whatwg/xhr] Resource Timing integration (#319)

@yoavweiss commented on this pull request.



> @@ -954,6 +954,10 @@ return <a>this</a>'s <a>cross-origin credentials</a>.
   </ol>
 </ol>
 
+<p>To <dfn>report timing</dfn> for an {{XMLHttpRequest}} object <var>xhr</var>,
+<a for=/>finalize and report timing</a> for <var>xhr</var>'s <a for=XMLHttpRequest>response</a>,

s/for/with/ ?

> @@ -963,6 +967,8 @@ return <a>this</a>'s <a>cross-origin credentials</a>.
  <li><p>If <var>xhr</var>'s <a for=XMLHttpRequest>response</a> is a <a>network error</a>, then
  return.
 
+ <li><p><a for=/>Report timing</a> for <var>xhr</var>.

Could we move this to line 966, and avoid calling report timing from 2 different places?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/xhr/pull/319#pullrequestreview-877248705
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/xhr/pull/319/review/877248705@github.com>

Received on Wednesday, 9 February 2022 10:35:50 UTC