Merge
authorvaleriep
Thu, 15 Mar 2012 14:40:53 -0700
changeset 12194 1a48716abaa1
parent 12193 8ad489a789e1 (current diff)
parent 12192 dd8301ce22b6 (diff)
child 12195 14f20316b1b8
Merge
--- 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);