[Webkit-unassigned] [Bug 86067] [BlackBerry] Possible to clobber httponly cookie.

[bugzilla-daemon at webkit.org]

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





--- Comment #6 from jochen at chromium.org  2012-05-18 12:21:55 PST ---
(From update of attachment 142673)
View in context: https://bugs.webkit.org/attachment.cgi?id=142673&action=review

> Source/WebCore/platform/blackberry/CookieManager.cpp:513
> +    getRawCookies(cookies, url, WithHttpOnlyCookies);

the only way I see this could be optimized is to move getRawCookies out of the loop. In the end, to insert a cookie, you'll have to check all cookies for the given url, so there's not really much to be improved

> LayoutTests/http/tests/cookies/js-get-and-set-http-only-cookie.php:17
> +

nit. no empty lines at eof

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