New issue
Advanced search Search tips

Issue 906538 link

Starred by 1 user

Issue metadata

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

Blocking:
issue 906536



Sign in to add a comment

Have a regression test for crbug.com/895999

Project Member Reported by toyoshim@chromium.org, Nov 19

Issue description

Blocking: 906536
Cc: toyoshim@chromium.org yhirano@chromium.org
Status: Assigned (was: Untriaged)
karandeepb, can you help us to have tests for this?
We are planning to enable OOR-CORS soon, and will have CORS checks for file URL scheme again for the new code path. In the new code path, this check is not optional, but mandatory.
Actually, ExtensionApiTest.FileURLs may be the test for this?
It fails with kOutOfBlinkCORS now.

But it may be not enough to detect the case of crbug/895999 because it passed even we have the yhirano's check.
Cc: karandeepb@chromium.org
Owner: toyoshim@chromium.org
I am quite swamped right now, so won't be able to get to this soon. Not sure why ExtensionApiTest.FileURLs didn't fail.

Assigning to toyoshim@ in the meanwhile.
Cc: devlin@chromium.org
Owner: ----
Status: Available (was: Assigned)
Can you find someone from extension experts?
It looks a little hard to add tests in this area for Blink people.

I'm currently focusing on making existing tests work to enable OOR-CORS by default. So, adding new tests would be the second priority for me at this moment.

Sign in to add a comment