[webkit-reviews] review granted: [Bug 234238] FetchResponse::clone should use the relevant realm for the cloned response : [Attachment 446991] Patch

Mon Dec 13 09:02:12 PST 2021

Alexey Shvayka <ashvayka at apple.com> has granted youenn fablet
<youennf at gmail.com>'s request for review:
Bug 234238: FetchResponse::clone should use the relevant realm for the cloned
response
https://bugs.webkit.org/show_bug.cgi?id=234238

Attachment 446991: Patch

https://bugs.webkit.org/attachment.cgi?id=446991&action=review

--- Comment #3 from Alexey Shvayka <ashvayka at apple.com> ---
Comment on attachment 446991
  --> https://bugs.webkit.org/attachment.cgi?id=446991
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=446991&action=review

This is marvelous! I was looking into Fetch stuff during
https://bugs.webkit.org/show_bug.cgi?id=230941 yet didn't come up with a test
case.

With that said, are there any fetch methods we can fix in the same way? Namely
the SEC FetchResponse's error() / redirect() are using to create another
response?

> LayoutTests/http/wpt/fetch/clone-realm.html:1
> +<!doctype html>

There is no way we can WPT export this test, right?