[Webkit-unassigned] [Bug 82041] [BlackBerry]Missing cookies from HTTP response header in Network tab of Web Inspector.

[bugzilla-daemon at webkit.org]

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





--- Comment #17 from Rob Buis <rwlbuis at gmail.com>  2012-03-28 04:17:03 PST ---
(From update of attachment 133958)
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?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.