[webkit-reviews] review denied: [Bug 99950] [BlackBerry] Missing some cookies in HTTP response header when set several cookies in one "Set-Cookie" header. : [Attachment 169825] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Oct 21 23:27:53 PDT 2012


George Staikos <staikos at kde.org> has denied Peter Wang
<peter.wang at torchmobile.com.cn>'s request for review:
Bug 99950: [BlackBerry] Missing some cookies in HTTP response header when set
several cookies in one "Set-Cookie" header.
https://bugs.webkit.org/show_bug.cgi?id=99950

Attachment 169825: Patch
https://bugs.webkit.org/attachment.cgi?id=169825&action=review

------- Additional Comments from George Staikos <staikos at kde.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=169825&action=review


Please make the comment clearer first.

> Source/WebCore/platform/network/blackberry/NetworkJob.cpp:338
> +	       // If there are several "Set-Cookie" headers, we should combine
the followed with headed.

"following ones with this header" is what is meant here?


More information about the webkit-reviews mailing list