--- a/jdk/make/common/shared/Sanity.gmk Thu Mar 15 14:28:48 2012 -0700
+++ b/jdk/make/common/shared/Sanity.gmk Thu Mar 15 14:40:53 2012 -0700
@@ -348,7 +348,13 @@
" Try setting LANG to 'C'. \n" \
"" >> $(WARNING_FILE) ; \
fi
+ifeq ($(PLATFORM), macosx)
+ @if [ "$(LANG)" = "" ]; then \
+ $(ECHO) "ERROR: LANG must be set on Mac OS X. Recommended value is \"C\"" >> $(ERROR_FILE) ; \
+ fi
endif
+endif
+
######################################################
# Check the Windows cygwin version
--- a/jdk/src/share/classes/java/net/InMemoryCookieStore.java Thu Mar 15 14:28:48 2012 -0700
+++ b/jdk/src/share/classes/java/net/InMemoryCookieStore.java Thu Mar 15 14:40:53 2012 -0700
@@ -207,6 +207,9 @@
public boolean removeAll() {
lock.lock();
try {
+ if (cookieJar.isEmpty()) {
+ return false;
+ }
cookieJar.clear();
domainIndex.clear();
uriIndex.clear();
--- a/jdk/test/java/net/CookieHandler/NullUriCookieTest.java Thu Mar 15 14:28:48 2012 -0700
+++ b/jdk/test/java/net/CookieHandler/NullUriCookieTest.java Thu Mar 15 14:40:53 2012 -0700
@@ -23,8 +23,9 @@
/*
* @test
- * @bug 6953455
+ * @bug 6953455 7045655
* @summary CookieStore.add() cannot handle null URI parameter
+ * and An empty InMemoryCookieStore should not return true for removeAll
*/
import java.net.CookieManager;
@@ -44,6 +45,11 @@
static void checkCookieNullUri() throws Exception {
//get a cookie store implementation and add a cookie to the store with null URI
CookieStore cookieStore = (new CookieManager()).getCookieStore();
+ //Check if removeAll() retrurns false on an empty CookieStore
+ if (cookieStore.removeAll()) {
+ fail = true;
+ }
+ checkFail("removeAll on empty store should return false");
HttpCookie cookie = new HttpCookie("MY_COOKIE", "MY_COOKIE_VALUE");
cookie.setDomain("foo.com");
cookieStore.add(null, cookie);