New issue
Advanced search Search tips

Issue 632062 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

CacheStorageCache::Close should prevent future operations from scheduling

Project Member Reported by jkarlin@chromium.org, Jul 27 2016

Issue description

CacheStorageCache::Close schedules a close operation. Between when the Close is scheduled and when it actually runs, more operations might be put on the queue. Those operations won't get a chance to run if Close's callback immediately deletes the cache object.

We should alter Close() to set a "closing_" variable which prevents future operations from scheduling, instead they'll immediately run their callbacks with an error state. Then it's safe to delete the CacheStorageCache object once Close() runs its callback.

 
Components: Blink>Storage>CacheStorage
Labels: OS-All
Status: Available (was: Untriaged)
Project Member

Comment 2 by sheriffbot@chromium.org, Jul 28 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by dmu...@chromium.org, Aug 11 2017

Status: Available (was: Untriaged)
Project Member

Comment 4 by sheriffbot@chromium.org, Aug 13

Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: wanderview@chromium.org pwnall@chromium.org
Status: Available (was: Untriaged)

Sign in to add a comment