RESOLVED FIXED Bug 188756
Remove experimental affiliated domain code now that StorageAccess API is available 
https://bugs.webkit.org/show_bug.cgi?id=188756
Summary Remove experimental affiliated domain code now that StorageAccess API is avai...
Brent Fulgham
Reported 2018-08-20 13:37:27 PDT
In Bug 174661 we added a compatibility quirk to support wsj.com authentication. This quirk is no longer needed, since the StorageAccess API provides the necessary tools to do this type of interaction without needing global cross-site access.
Attachments
Patch (6.34 KB, patch)
2018-08-20 14:33 PDT, Brent Fulgham 		no flags
DetailsFormatted DiffDiff
Patch (6.77 KB, patch)
2018-08-20 17:11 PDT, Brent Fulgham 		no flags
DetailsFormatted DiffDiff
Patch (14.38 KB, patch)
2018-08-21 09:28 PDT, Brent Fulgham 		no flags
DetailsFormatted DiffDiff
Patch (13.92 KB, patch)
2018-08-21 10:45 PDT, Brent Fulgham 		no flags
DetailsFormatted DiffDiff
Patch (14.08 KB, patch)
2018-08-21 11:40 PDT, Brent Fulgham 		achristensen: review+
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Brent Fulgham
Comment 1 2018-08-20 14:30:56 PDT
<rdar://problem/43527848>
Brent Fulgham
Comment 2 2018-08-20 14:33:59 PDT
Created attachment 347546 [details] Patch
Brent Fulgham
Comment 3 2018-08-20 17:11:46 PDT
Created attachment 347576 [details] Patch
Brent Fulgham
Comment 4 2018-08-21 09:28:51 PDT
Created attachment 347648 [details] Patch
John Wilander
Comment 5 2018-08-21 10:26:05 PDT
Mac debug test failures are unrelated.
Brent Fulgham
Comment 6 2018-08-21 10:26:43 PDT
Despite Alex's r+, I think I made a mistake. I'll upload a revised version.
John Wilander
Comment 7 2018-08-21 10:28:45 PDT
You probably want to merge with the queued patch from https://bugs.webkit.org/show_bug.cgi?id=188757 or rebase when it's landed.
Brent Fulgham
Comment 8 2018-08-21 10:45:24 PDT
Created attachment 347659 [details] Patch
Brent Fulgham
Comment 9 2018-08-21 11:40:54 PDT
Created attachment 347664 [details] Patch
Brent Fulgham
Comment 10 2018-08-21 11:41:33 PDT
Revised patch merged with current ToT uploaded. Ready for a new review!
John Wilander
Comment 11 2018-08-21 13:10:57 PDT
Comment on attachment 347664 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=347664&action=review Looks good to me. > Source/WebCore/loader/ResourceLoadObserver.cpp:150 > + Ah, this is a fix of the comparison that was lost when I removed the call to areDomainsAssociated(). Good catch. > Source/WebKit/UIProcess/ResourceLoadStatisticsMemoryStore.cpp:611 > + I see, you moved the previous check in WebResourceLoadStatisticsStore's areDomainsAssociated() into here.
Brent Fulgham
Comment 12 2018-08-21 13:37:34 PDT
Committed r235132: <https://trac.webkit.org/changeset/235132>
Note You need to log in before you can comment on or make changes to this bug.