HtmlUnitDriver.deleteAllCookies() fix

WebDriver.deleteAllCookies() deletes all the cookies
for the current domain, however HtmlUnitDriver implementation
deletes all cookies for all domains, which is incorrect.

This was pointed out in
https://sourceforge.net/p/htmlunit/bugs/1695/

Test case attached as well

Signed-off-by: Luke Inman-Semerau <[email protected]>

Author: asashour

Diff for: java/client/src/org/openqa/selenium/htmlunit/HtmlUnitDriver.java

@@ -1501,7 +1501,13 @@ public void deleteCookie(Cookie cookie) {
 
     @Override
     public void deleteAllCookies() {
-      getWebClient().getCookieManager().clearCookies();
+      CookieManager cookieManager = getWebClient().getCookieManager();
+
+      URL url = getRawUrl();
+      Set<com.gargoylesoftware.htmlunit.util.Cookie> rawCookies = getWebClient().getCookies(url);
+      for (com.gargoylesoftware.htmlunit.util.Cookie cookie : rawCookies) {
+          cookieManager.removeCookie(cookie);
+      }
     }
 
     @Override

Diff for: java/client/test/org/openqa/selenium/CookieImplementationTest.java

@@ -572,4 +572,35 @@ private void addCookieOnServerSide(Cookie cookie) {
     }
     driver.get(url.toString());
   }
+
+  @Test
+  public void deleteAllCookies() throws Exception {
+    Cookie cookie1 = new Cookie.Builder("fish1", "cod")
+        .domain(appServer.getHostName()).build();
+    Cookie cookie2 = new Cookie.Builder("fish2", "tune")
+        .domain(appServer.getAlternateHostName()).build();
+
+    String url1 = domainHelper.getUrlForFirstValidHostname("/common");
+    String url2 = domainHelper.getUrlForSecondValidHostname("/common");
+
+    WebDriver.Options options = driver.manage();
+
+    options.addCookie(cookie1);
+    assertCookieIsPresentWithName(cookie1.getName());
+
+    driver.get(url2);
+    options.addCookie(cookie2);
+    assertCookieIsNotPresentWithName(cookie1.getName());
+    assertCookieIsPresentWithName(cookie2.getName());
+
+    driver.get(url1);
+    assertCookieIsPresentWithName(cookie1.getName());
+    assertCookieIsNotPresentWithName(cookie2.getName());
+
+    options.deleteAllCookies();
+    assertCookieIsNotPresentWithName(cookie1.getName());
+
+    driver.get(url2);
+    assertCookieIsPresentWithName(cookie2.getName());
+  }
 }