Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(583)

Issue 528283002: [WIP]Ignore cache on reloading CORS-involved resources.

Created:
6 years, 3 months ago by sof
Modified:
6 years, 3 months ago
Reviewers:
CC:
blink-reviews, Nate Chapin, gavinp+loader_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Visibility:
Public.

Description

[WIP]Ignore cache on reloading CORS-involved resources. If a resource is cached in the local memory cache, but was previously fetched with a differing CORS setting, the resource fetcher cannot reuse the resource, purging it and issuing a reload request instead. Arrange for that reload to also ignore whatever is cached on the browser process' side also, ensuring that the resource fetch gets a "CORS consistent" response. R= BUG=409090

Patch Set 1 #

Patch Set 2 : Add test #

Patch Set 3 : Add helper php script #

Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -17 lines) Patch
A LayoutTests/http/tests/images/image-with-and-without-origin-header.html View 1 1 chunk +37 lines, -0 lines 0 comments Download
A + LayoutTests/http/tests/images/image-with-and-without-origin-header-expected.txt View 1 1 chunk +2 lines, -2 lines 0 comments Download
A LayoutTests/http/tests/resources/origin-image.php View 1 2 1 chunk +14 lines, -0 lines 0 comments Download
M Source/core/fetch/ResourceFetcher.h View 1 chunk +6 lines, -5 lines 0 comments Download
M Source/core/fetch/ResourceFetcher.cpp View 1 9 chunks +15 lines, -10 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698