New issue
Advanced search Search tips

Issue 777517 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Make all cache hits in blink async

Project Member Reported by japhet@chromium.org, Oct 23 2017

Issue description

By default, when a Resource is reused from blink's MemoryCache, a task is posted to return the cached data.

https://chromium.googlesource.com/chromium/src/+/1d17db84433d846eb3ad0b48fe6b7b749cae6fc2/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp#645 defines a set of Resource::Type values that are exempt from this and process the cache hit synchronously.

This is a tracking bug for working through these cases to try to get rid of the special cases, and to follow up on any regressions that might result.
 

Comment 1 by japhet@chromium.org, Oct 23 2017

Status: WontFix (was: Assigned)
Oops, forgot I already had https://bugs.chromium.org/p/chromium/issues/detail?id=237366 open :)
Nate, are you also interested in static resources, such as issue 771425?

Comment 3 by japhet@chromium.org, Oct 23 2017

That would be part of 237366, yes. Having CSSStylesheet return false in https://chromium.googlesource.com/chromium/src/+/1d17db84433d846eb3ad0b48fe6b7b749cae6fc2/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp#645 would fix 771425 as well.

Sign in to add a comment