Bug 169756

Summary:

Add a reload policy where only expired subresources are revalidated

Product:

WebKit

Reporter:

Antti Koivisto <koivisto>

Component:

Page Loading

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

beidson, berto, cdumez, cgarcia, commit-queue, dbates, gustavo, japhet, mcatanzaro

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
patch	none
patch	none
patch	none
patch	kling: review+
patch	none

Antti Koivisto

Reported 2017-03-16 08:41:11 PDT

Currently a reload will revalidate all subresources on the page (except those with Cache-Control: immutable). Add a weaker policy where only expired subresources are revalidated.

Attachments
patch (39.62 KB, patch) 2017-03-17 06:29 PDT, Antti Koivisto	no flags	Details Formatted Diff Diff
patch (39.59 KB, patch) 2017-03-17 06:54 PDT, Antti Koivisto	no flags	Details Formatted Diff Diff
patch (39.59 KB, patch) 2017-03-17 07:16 PDT, Antti Koivisto	no flags	Details Formatted Diff Diff
patch (39.56 KB, patch) 2017-03-17 07:18 PDT, Antti Koivisto	kling: review+	Details Formatted Diff Diff
patch (40.92 KB, patch) 2017-03-17 07:54 PDT, Antti Koivisto	no flags	Details Formatted Diff Diff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.

Antti Koivisto

Comment 1 2017-03-17 06:29:47 PDT

Created attachment 304772 [details] patch

Antti Koivisto

Comment 2 2017-03-17 06:54:20 PDT

Created attachment 304774 [details] patch

Antti Koivisto

Comment 3 2017-03-17 07:16:01 PDT

Created attachment 304775 [details] patch

Antti Koivisto

Comment 4 2017-03-17 07:18:45 PDT

Created attachment 304776 [details] patch

Andreas Kling

Comment 5 2017-03-17 07:47:01 PDT

Comment on attachment 304775 [details] patch r=me

Antti Koivisto

Comment 6 2017-03-17 07:54:45 PDT

Created attachment 304777 [details] patch

WebKit Commit Bot

Comment 7 2017-03-17 07:55:50 PDT

Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API

WebKit Commit Bot

Comment 8 2017-03-17 12:52:53 PDT

Comment on attachment 304777 [details] patch Clearing flags on attachment: 304777 Committed r214113: <http://trac.webkit.org/changeset/214113>

WebKit Commit Bot

Comment 9 2017-03-17 12:52:58 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.