[webkit-reviews] review granted: [Bug 195196] Resource Load Statistics: Further restrict client-side cookie persistence after cross-site navigations with link decoration : [Attachment 363290] Patch

[bugzilla-daemon at webkit.org]

Brent Fulgham <bfulgham at webkit.org> has granted John Wilander
<wilander at apple.com>'s request for review:
Bug 195196: Resource Load Statistics: Further restrict client-side cookie
persistence after cross-site navigations with link decoration
https://bugs.webkit.org/show_bug.cgi?id=195196

Attachment 363290: Patch

https://bugs.webkit.org/attachment.cgi?id=363290&action=review




--- Comment #3 from Brent Fulgham <bfulgham at webkit.org> ---
Comment on attachment 363290
  --> https://bugs.webkit.org/attachment.cgi?id=363290
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=363290&action=review

The debug failures exist without this patch, so are unrelated.

r=me

> Source/WebCore/platform/network/NetworkStorageSession.h:183
> +    Optional<Seconds> clientSideCookieCap(const RegistrableDomain&
firstParty, Optional<uint64_t> pageID) const;

Optional<PageID>?