Evict "fallback" RawResource from MemoryCache

Evict "fallback" RawResource from MemoryCache

A service worker may ask the original renderer to re-fetch the request when a
fetch-mode=CORS request was not handled. In most cases ResourceLoader restarts
the resource, but in some cases it lets DocumentThreadableLoader handle the
response. In such a case, we can see a Resource whose response's
WasFallbackRequiredByServiceWorker is true.

Previously ResourceFetcher::DetermineRevalidationPolicy forbade to reuse such
a resource. This CL removes the logic. Instead, RawResource evicts itself when
receiving such a response.

BUG=652228
R=horo@chromium.org, tyoshino@chromium.org

Review-Url: https://codereview.chromium.org/2830753003
Cr-Commit-Position: refs/heads/master@{#465987}
Committed: https://chromium.googlesource.com/chromium/src/+/5705c2783128e015cf8c9da3f7a1b188d81101b8

[yhirano, 2017-04-20 08:27:18]

The CQ bit was checked by yhirano@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-20 08:27:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-20 09:19:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-20 09:19:09]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[horo, 2017-04-20 10:22:32]

lgtm

[tyoshino (SeeGerritForStatus), 2017-04-20 12:03:37]

CL description: Previsouly -> Previously

[yhirano, 2017-04-20 12:04:53]

Description was changed from

==========
Evict "fallback" RawResource from MemoryCache

A service worker may ask the original renderer to re-fetch the request when a
fetch-mode=CORS request was not handled. In most cases ResourceLoader restarts
the resource, but in some cases it lets DocumentThreadableLoader handle the
response. In such a case, we can see a Resource whose response's
WasFallbackRequiredByServiceWorker is true.

Previsouly ResourceFetcher::DetermineRevalidationPolicy forbade to reuse such
a resource. This CL removes the logic. Instead, RawResource evicts itself when
receiving such a response.

BUG=652228
R=horo@chromium.org, tyoshino@chromium.org
==========

to

==========
Evict "fallback" RawResource from MemoryCache

A service worker may ask the original renderer to re-fetch the request when a
fetch-mode=CORS request was not handled. In most cases ResourceLoader restarts
the resource, but in some cases it lets DocumentThreadableLoader handle the
response. In such a case, we can see a Resource whose response's
WasFallbackRequiredByServiceWorker is true.

Previously ResourceFetcher::DetermineRevalidationPolicy forbade to reuse such
a resource. This CL removes the logic. Instead, RawResource evicts itself when
receiving such a response.

BUG=652228
R=horo@chromium.org, tyoshino@chromium.org
==========

[yhirano, 2017-04-20 12:05:00]

On 2017/04/20 12:03:37, tyoshino wrote:
> CL description: Previsouly -> Previously

Done.

[tyoshino (SeeGerritForStatus), 2017-04-20 12:14:15]

lgtm

[yhirano, 2017-04-20 12:14:45]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2017-04-20 12:15:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-20 12:52:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492690485106730, "parent_rev":
"349c2bfb9b604092ff8c4a78752cc7e25a1fca34", "commit_rev":
"5705c2783128e015cf8c9da3f7a1b188d81101b8"}

[commit-bot: I haz the power, 2017-04-20 12:52:25]

Description was changed from

==========
Evict "fallback" RawResource from MemoryCache

A service worker may ask the original renderer to re-fetch the request when a
fetch-mode=CORS request was not handled. In most cases ResourceLoader restarts
the resource, but in some cases it lets DocumentThreadableLoader handle the
response. In such a case, we can see a Resource whose response's
WasFallbackRequiredByServiceWorker is true.

Previously ResourceFetcher::DetermineRevalidationPolicy forbade to reuse such
a resource. This CL removes the logic. Instead, RawResource evicts itself when
receiving such a response.

BUG=652228
R=horo@chromium.org, tyoshino@chromium.org
==========

to

==========
Evict "fallback" RawResource from MemoryCache

A service worker may ask the original renderer to re-fetch the request when a
fetch-mode=CORS request was not handled. In most cases ResourceLoader restarts
the resource, but in some cases it lets DocumentThreadableLoader handle the
response. In such a case, we can see a Resource whose response's
WasFallbackRequiredByServiceWorker is true.

Previously ResourceFetcher::DetermineRevalidationPolicy forbade to reuse such
a resource. This CL removes the logic. Instead, RawResource evicts itself when
receiving such a response.

BUG=652228
R=horo@chromium.org, tyoshino@chromium.org

Review-Url: https://codereview.chromium.org/2830753003
Cr-Commit-Position: refs/heads/master@{#465987}
Committed:
https://chromium.googlesource.com/chromium/src/+/5705c2783128e015cf8c9da3f7a1...
==========

[commit-bot: I haz the power, 2017-04-20 12:52:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5705c2783128e015cf8c9da3f7a1...