New issue
Advanced search Search tips

Issue 911959 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

A=>B=>A redirect for "no-cors" should result in "opaque" response type, not "basic".

Project Member Reported by yhirano@chromium.org, Dec 5

Issue description

Cc: annevank...@gmail.com
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 5

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/73b133f94f1f352f2b4151ff4b0c7c1e58f88afd

commit 73b133f94f1f352f2b4151ff4b0c7c1e58f88afd
Author: Yutaka Hirano <yhirano@chromium.org>
Date: Wed Dec 05 13:52:40 2018

A=>B=>A redirect for "no-cors" should result in "opaque" response type

This CL follows https://github.com/whatwg/fetch/pull/834. This CL
changes services/network only because I accidentally did the same thing
for blink::ResourceLoader, and what blink::ThreadableLoader does is
not important because blink::FetchManager has its own response type
setting logic.

Bug: 911959

Change-Id: I62f6ea9401e4fd7d87d01f6cb021feef8e06bb88
Reviewed-on: https://chromium-review.googlesource.com/c/1362774
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#613956}
[modify] https://crrev.com/73b133f94f1f352f2b4151ff4b0c7c1e58f88afd/services/network/cors/cors_url_loader.cc
[modify] https://crrev.com/73b133f94f1f352f2b4151ff4b0c7c1e58f88afd/services/network/public/cpp/cors/cors.cc
[modify] https://crrev.com/73b133f94f1f352f2b4151ff4b0c7c1e58f88afd/services/network/public/cpp/cors/cors.h
[modify] https://crrev.com/73b133f94f1f352f2b4151ff4b0c7c1e58f88afd/services/network/public/cpp/cors/cors_unittest.cc
[modify] https://crrev.com/73b133f94f1f352f2b4151ff4b0c7c1e58f88afd/third_party/blink/renderer/platform/loader/cors/cors.cc

Sign in to add a comment