Closed Bug 963703 Opened 6 years ago Closed 6 years ago

HTTP cache v2: Don't use LoadUnblocked flag as cache priority load indication

Categories

(Core :: Networking: Cache, defect)

Other Branch
x86_64
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: mayhemer, Assigned: mayhemer)

References

Details

Attachments

(1 file)

According how this flag is defined and used (mainly for XHR to not be blocked on head css/js loads) it is not a perf win to consider them as a cache priority loads.  Cache is not blocked by any priority loading, so only affect is that the XHR will load a bit later, while expected to not have any major impact on page load and user experience performance.
Attached patch v1Splinter Review
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Attachment #8365259 - Flags: review?(michal.novotny)
Michal, this should land before we do the temporary m-c testing.  Can give the one line-review a priority please?
Attachment #8365259 - Flags: review?(michal.novotny) → review+
https://hg.mozilla.org/mozilla-central/rev/0eb2ba65a702
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
No longer blocks: 957243
You need to log in before you can comment on or make changes to this bug.