Thanks Kurchi, the change look good to me.
This is a corner case and already covered by JCK, but it may be useful
to amend an existing test to check for this. Maybe
test/java/net/CookieHandler/NullUriCookieTest.java
CookieStore cookieStore = (new CookieManager()).getCookieStore();
+
Hi Chris,
Updated webrev: http://cr.openjdk.java.net/~khazra/7045655/webrev.02/
- Kurchi
On 3/15/2012 2:49 AM, Chris Hegarty wrote:
Thanks Kurchi, the change look good to me.
This is a corner case and already covered by JCK, but it may be useful
to amend an existing test to check for
Thanks Kurchi, looks good.
-Chris.
Kurchi Hazra kurchi.subhra.ha...@oracle.com wrote:
Hi Chris,
Updated webrev: http://cr.openjdk.java.net/~khazra/7045655/webrev.02/
- Kurchi
On 3/15/2012 2:49 AM, Chris Hegarty wrote:
Thanks Kurchi, the change look good to me.
This is a corner case
The CookieStore.removeAll() is supposed to return true according to its
spec, only if the CookieStore changes as a
result of the call.
InMemoryCookieStore:removeAll() was returning true by default, even if
the CookieStore object was already empty,
and no changes were being done by the call.