Bug 185985

Summary: Avoid triggering network cache speculative revalidation for loads allowing expired content
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: Page LoadingAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: beidson, cgarcia, commit-queue, ews-watchlist, ggaren, koivisto, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Chris Dumez
Reported 2018-05-25 09:46:41 PDT
Avoid triggering network cache speculative revalidation for loads allowing expired content (e.g. history loads). This causes us to do unnecessary revalidations.
Attachments
Patch (1.87 KB, patch)
2018-05-25 09:49 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2018-05-25 09:49:18 PDT
Created attachment 341296 [details] Patch
Geoffrey Garen
Comment 2 2018-05-25 09:58:23 PDT
If we skip speculative revalidation, does that mean that we'll also skip the optimization that brings all the resources for a given URL into memory?
WebKit Commit Bot
Comment 3 2018-05-25 10:23:19 PDT
Comment on attachment 341296 [details] Patch Clearing flags on attachment: 341296 Committed r232190: <https://trac.webkit.org/changeset/232190>
WebKit Commit Bot
Comment 4 2018-05-25 10:23:21 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 5 2018-05-25 10:24:55 PDT
<rdar://problem/40557978>
Note You need to log in before you can comment on or make changes to this bug.