Summary: | Resource Load Statistics: Flip experimental cookie blocking setting from an enable to a disable | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | John Wilander <wilander> | ||||
Component: | WebKit Misc. | Assignee: | John Wilander <wilander> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bfulgham, commit-queue, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Local Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
John Wilander
2020-01-08 15:28:09 PST
Created attachment 387149 [details]
Patch
Comment on attachment 387149 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=387149&action=review > Source/WebKit/UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:75 > + auto thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::All; When would we ever hit 'OnlyAccordingToPerDomainPolicy'? (In reply to Brent Fulgham from comment #3) > Comment on attachment 387149 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=387149&action=review > > > Source/WebKit/UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:75 > > + auto thirdPartyCookieBlockingMode = WebCore::ThirdPartyCookieBlockingMode::All; > > When would we ever hit 'OnlyAccordingToPerDomainPolicy'? It was the old default but now it's only needed to be able to run layout tests with non-ITP behavior by default. The TestRunner falls back to this old setting for all tests that don't explicitly test ITP's behavior. Otherwise a bunch of tests that assume cookie access would not pass. My reasoning is that since users can turn off ITP and WKWebViews don't run with ITP, the rest of the layout test suite needs to continue testing under such conditions. (Per-domain policy effectively means that if the layout test doesn't make a domain prevalent, ITP won't do anything, ergo non-ITP behavior.) I hope this makes sense. Thanks for the review! Comment on attachment 387149 [details] Patch Clearing flags on attachment: 387149 Committed r254239: <https://trac.webkit.org/changeset/254239> All reviewed patches have been landed. Closing bug. |