[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 #29 from jochen at chromium.org  2012-06-04 07:14:55 PST ---
(From update of attachment 145520)
In order to successfully contribute to WebKit, you should be able to build a few other ports..

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

> LayoutTests/platform/blackberry/http/tests/cookies/js-get-and-set-http-only-cookie.php:2
> +setcookie("httpOnlyCookie", "value", time()+36000000, "", "", 0, 1);

you should use the cookie scripts located in http/tests/cookies/resources as in your previos patch to set the cookie.

The bug in your previous patch was that you use php to set the cookie, instead of the setCookie helper function. Therefore, the call to clearCookies() had no effect.

Please revert to the previous version, but change your test to not use PHP to set the cookie. Please also remove the baselines committed by other ports (such as LayoutTests/platform/mac/http/tests/cookies/js-get-and-set-http-only-cookie-expected.txt)

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