[Webkit-unassigned] [Bug 82041] [BlackBerry]Missing cookies from HTTP response header in Network tab of Web Inspector.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Mar 28 19:07:17 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=82041
--- Comment #18 from Jason <jason.liu at torchmobile.com.cn> 2012-03-28 19:07:17 PST ---
(In reply to comment #17)
> (From update of attachment 133958 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=133958&action=review
>
> > Source/WebCore/ChangeLog:6
> > + We shouldn't overwrite cookies of response if there are more than one
>
> Don't you mean "one or more"?
>
> > Source/WebCore/platform/network/blackberry/NetworkJob.cpp:318
> > + if (m_response.httpHeaderFields().contains("Set-Cookie")) {
>
> Is there an AtomicString/static String for Set-Cookie?
>
> > Source/WebCore/platform/network/blackberry/NetworkJob.cpp:319
> > + m_response.setHTTPHeaderField(key, m_response.httpHeaderField(key) + "\r\n" + value);
>
> Is "\r\n" used like that all the time? Any constants?
1. I mean the second cookie shouldn't overwrite the first one of response.
2. No, there isn't.
3. "\r\n" is only used in this special case.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list