Bug 36997 – LayoutTests/http/tests/cookies/double-quoted-value-with-semi-colon.html should not use a new-style cookie

• Status: NEW
• Component: Tools / Tests

[David Kilzer (:ddkilzer)]

* SUMMARY
LayoutTests/http/tests/cookies/double-quoted-value-with-semi-colon.html uses a "Version=1", new-style cookie (as specified in <http://www.ietf.org/rfc/rfc2109.txt>).  However, the cookie testing code doesn't support this type of cookie yet, so it should be changed back to an original-style cookie.

[David Kilzer (:ddkilzer)]

I filed Bug 36998 to add support for testing RFC-2109-style cookies.

[David Kilzer (:ddkilzer)]

Created attachment 52359 [details]
Patch v1

[David Kilzer (:ddkilzer)]

<rdar://problem/7767948>

[Eric Seidel (no email)]

Adam says this test is wrong.  I've CC'd him and he can explain why.

[Adam Barth]

This cookie isn't correct according to the new cookie spec:

https://datatracker.ietf.org/doc/draft-ietf-httpstate-cookie/

Quoted cookie values were a fantasy invented by 2109 to cause pain in the world.

[Adam Barth]

Comment on attachment 52359 [details]
Patch v1

The line of code should read:

cookiesShouldBe('disorder="477beccb;richard";Path=/', 'disorder="477beccb');

I'd rather we tested non-esoteric properties of cookies before testing this particular case (which is wrong in CFNetwork).

[David Kilzer (:ddkilzer)]

Perhaps we should simply remove the test.

Is there a related test in your test suite?

I've updated Bug 36998 for importing your cookie test suite rather than writing tests for RFC 2109.