Bug 140302

Summary: Allow HTTPS + 'Cache-control: no-store' sub-frames into the page cache
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: Page LoadingAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, barraclough, commit-queue, joepeck, kling, koivisto
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Chris Dumez 2015-01-09 11:33:48 PST 
We should allow HTTPS + 'Cache-control: no-store' sub-frames into the page cache. We already restore 'no-store' sub-resources on history navigation from the memory cache so there is no reason for our page cache policy to be more restrictive.
We should align our memory cache / history navigation policy with our page cache policy.

For now, 'no-store' main resources are not restored from either cache (memory cache / page cache) on history navigation though. This behavior does not change.
Comment 1 Chris Dumez 2015-01-09 11:39:24 PST 
Created attachment 244355 [details]
Patch
Comment 2 Andreas Kling 2015-01-09 12:11:36 PST 
Comment on attachment 244355 [details]
Patch

r=me
Comment 3 Chris Dumez 2015-01-09 12:19:34 PST 
Created attachment 244364 [details]
Patch
Comment 4 WebKit Commit Bot 2015-01-09 12:59:06 PST 
Comment on attachment 244364 [details]
Patch

Clearing flags on attachment: 244364

Committed r178191: <http://trac.webkit.org/changeset/178191>
Comment 5 WebKit Commit Bot 2015-01-09 12:59:11 PST 
All reviewed patches have been landed.  Closing bug.