New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 759960 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

MemoryCache should not cache data: URLs

Project Member Reported by vdje...@fb.com, Aug 29 2017

Issue description

data: URLs waste space in the already space-limited (see issue 759958) memory cache.
They should not be cached at all.
 
Cc: kinuko@chromium.org hirosh...@chromium.org
Thanks for the report.

+Hiroshige, +Kinuko: any thoughts on this? If it makes sense, is it easy to make the change (i.e. do we need to spend time estimating the impact to justify the work)?

Comment 2 by kouhei@chromium.org, Sep 27 2017

Cc: japhet@chromium.org
Labels: Needs-Feedback
I'd like to understand more background on this.

MemoryCache doesn't hold any strong references to the resources now.
Also, MemoryCache contains Resource, which contains decoded content.
The fact that the content can be easily fetched again w/ low cost isn't enough to justify (svg images are small in raw bytes, but has non trivial decoding/rendering cost).
Status: WontFix (was: Untriaged)

Sign in to add a comment