Re: [whatwg/fetch] Http cache (#496)

yoavweiss commented on this pull request.



>  
- <li>
-  <p>If <var>response</var> is null, run these substeps:
+   <li>
+    <p>If the <var>revalidatingFlag</var> is set, then:

If I'm reading this correctly, nothing assigns `storedResponse` to `response` if revalidation is not needed, and the cache-mode is not `force-cache` or `only-if-cached`.
Am I reading this correctly? If so, is this intentional? I'd expect such assignment to happen if no revalidation is needed.

-- 
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/496#pullrequestreview-28099559