[Webkit-unassigned] [Bug 142770] Disk cache should support Vary: Cookie

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=142770

--- Comment #5 from Pratik Solanki <psolanki at apple.com> ---
Comment on attachment 248804
  --> https://bugs.webkit.org/attachment.cgi?id=248804
patch

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

> Source/WebKit2/ChangeLog:9
> +        Fetch then explicitly when resolving Vary: Cookie.

then -> them

> Source/WebKit2/ChangeLog:13
> +        reasonable cases though. Fetching cookies for every request might be too expensive for this rarely used feature.

Would it be worth having an assert to catch cases where cookies change in between? i.e. fetch the cookies for every request in debug builds and compare to what we saved.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150317/9e13f8d4/attachment-0002.html>