741678 - Opening of offline cache entry for write delays reading of entry from the nomal cache and/or from the net

Status: RESOLVED FIXED

(Core :: Networking: HTTP, enhancement)

Description

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

Our logic is something like this:

* Open a normal cache entry.
* If offline caching of the resource was requested, then open the offline cache
* Validate the normal cache entry, if any.
* Hit the net if necessary.

The logic should be reordered thusly:
* Open a normal cache entry.
* Validate the normal cache entry, if any.
* Hit the net if necessary.
* If we have something to write to the offline cache, and offline caching of the resource was requested, then open the offline cache

My understanding is that it is rare for an entry to be marked as needing to be written to the offline cache, so this shouldn't be a high priority relative to the other cache performance bugs.

Comment 1

[Honza Bambas (:mayhemer)]

This doesn't have IMO a high priority.  This (writing to offline cache) is done in background during the offline app cache update process and doesn't have a high probability of interacting with other requests in bursts during e.g. a front tab page load.

Comment 2

[Honza Bambas (:mayhemer)]

Fixed by 913807.