Bug 208387

Summary:

Adopt HTTP Alternative Services Storage

Product:

WebKit

Reporter:

jmehta

Component:

Page Loading

Assignee:

Alex Christensen <achristensen>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, beidson, benjamin, cdumez, cmarcelo, commit-queue, ews-watchlist, mjs, webkit-bug-importer, wilander

Priority:

Keywords:

InRadar

Version:

Other

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Proposed changes for HTTP Alternative Services	none
Proposed changes for HTTP Alternative Services	none
Patch	none
patch	none
Patch	none
Patch	none

jmehta

Reported 2020-02-28 12:53:46 PST

rdar://problem/56994565

Attachments
Proposed changes for HTTP Alternative Services (15.98 KB, patch) 2020-02-28 12:57 PST, jmehta	no flags	Details Formatted Diff Diff
Proposed changes for HTTP Alternative Services (15.99 KB, patch) 2020-02-28 13:58 PST, jmehta	no flags	Details Formatted Diff Diff
Patch (31.53 KB, patch) 2020-03-02 16:23 PST, Alex Christensen	no flags	Details Formatted Diff Diff
patch (31.53 KB, patch) 2020-03-02 17:06 PST, Alex Christensen	no flags	Details Formatted Diff Diff
Patch (35.05 KB, patch) 2020-03-02 23:31 PST, Alex Christensen	no flags	Details Formatted Diff Diff
Patch (39.88 KB, patch) 2020-03-03 09:34 PST, Alex Christensen	no flags	Details Formatted Diff Diff
Show Obsolete (5) View All Add attachment proposed patch, testcase, etc.

jmehta

Comment 1 2020-02-28 12:57:05 PST

Created attachment 392007 [details] Proposed changes for HTTP Alternative Services

jmehta

Comment 2 2020-02-28 13:58:18 PST

Created attachment 392016 [details] Proposed changes for HTTP Alternative Services

Maciej Stachowiak

Comment 3 2020-02-29 14:17:15 PST

Besides I breaking the build, we need to verify that this doesn’t create a potential supercookie like HSTS did, that it’s cleared by clearing website data, etc.

Alex Christensen

Comment 4 2020-02-29 14:19:19 PST

Comment on attachment 392016 [details] Proposed changes for HTTP Alternative Services This needs a new HAVE_ALTERNATIVE_SERVICES macro in Platform.h to not break the build. We will also need SPI on _WKWebsiteDataStoreConfiguration to specify the storage file location.

Alex Christensen

Comment 5 2020-03-02 16:23:27 PST

Created attachment 392211 [details] Patch

Alex Christensen

Comment 6 2020-03-02 17:06:07 PST

Created attachment 392220 [details] patch

Alex Christensen

Comment 7 2020-03-02 23:31:23 PST

Created attachment 392251 [details] Patch

Alex Christensen

Comment 8 2020-03-03 09:34:55 PST

Created attachment 392285 [details] Patch