chore: update driver to 1.23.0 (#968)

Author: yury-s

README.md

@@ -11,7 +11,7 @@ Playwright is a Java library to automate [Chromium](https://www.chromium.org/Hom
 
 |          | Linux | macOS | Windows |
 |   :---   | :---: | :---: | :---:   |
-| Chromium <!-- GEN:chromium-version -->103.0.5060.53<!-- GEN:stop --> | :white_check_mark: | :white_check_mark: | :white_check_mark: |
+| Chromium <!-- GEN:chromium-version -->104.0.5112.20<!-- GEN:stop --> | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 | WebKit <!-- GEN:webkit-version -->15.4<!-- GEN:stop --> | ✅ | ✅ | ✅ |
 | Firefox <!-- GEN:firefox-version -->100.0.2<!-- GEN:stop --> | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 

playwright/src/main/java/com/microsoft/playwright/Request.java

@@ -58,8 +58,9 @@ public interface Request {
    */
   Frame frame();
   /**
-   * **DEPRECATED** Incomplete list of headers as seen by the rendering engine. Use {@link Request#allHeaders
-   * Request.allHeaders()} instead.
+   * An object with the request HTTP headers. The header names are lower-cased. Note that this method does not return
+   * security-related headers, including cookie-related ones. You can use {@link Request#allHeaders Request.allHeaders()} for
+   * complete list of headers that include {@code cookie} information.
    */
   Map<String, String> headers();
   /**

playwright/src/main/java/com/microsoft/playwright/Response.java

@@ -45,8 +45,9 @@ public interface Response {
    */
   boolean fromServiceWorker();
   /**
-   * **DEPRECATED** Incomplete list of headers as seen by the rendering engine. Use {@link Response#allHeaders
-   * Response.allHeaders()} instead.
+   * An object with the response HTTP headers. The header names are lower-cased. Note that this method does not return
+   * security-related headers, including cookie-related ones. You can use {@link Response#allHeaders Response.allHeaders()}
+   * for complete list of headers that include {@code cookie} information.
    */
   Map<String, String> headers();
   /**

playwright/src/test/java/com/microsoft/playwright/TestBrowserContextCookies.java

@@ -65,13 +65,19 @@ void shouldGetANonSessionCookie() {
       "    return document.cookie;\n" +
       "  }");
     assertEquals("username=John Doe", documentCookie);
-    int timestamp = (Integer) page.evaluate("+(new Date('1/1/2038'))/1000");
     Cookie cookie = context.cookies().get(0);
     assertEquals("username", cookie.name);
     assertEquals("John Doe", cookie.value);
     assertEquals("localhost", cookie.domain);
     assertEquals("/", cookie.path);
-    assertEquals(timestamp, cookie.expires);
+    // Browsers start to cap cookies with 400 days max expires value.
+    // See https://github.com/httpwg/http-extensions/pull/1732
+    // Chromium patch: https://chromium.googlesource.com/chromium/src/+/aaa5d2b55478eac2ee642653dcd77a50ac3faff6
+    // We want to make sure that expires date is at least 400 days in future.
+    Double timestamp = (Double) page.evaluate("const FOUR_HUNDRED_DAYS = 1000 * 60 * 60 * 24 * 400;\n" +
+      "  const FIVE_MINUTES = 1000 * 60 * 5; // relax condition a bit to make sure test is not flaky.\n" +
+      "  (Date.now() + FOUR_HUNDRED_DAYS - FIVE_MINUTES) / 1000;");
+    assertTrue(cookie.expires > timestamp, cookie.expires + " > " + timestamp + " failed.");
     assertEquals(false, cookie.httpOnly);
     assertEquals(false, cookie.secure);
     if (isChromium()) {

playwright/src/test/java/com/microsoft/playwright/TestBrowserContextHar.java

@@ -19,6 +19,7 @@
 import com.microsoft.playwright.options.HarMode;
 import com.microsoft.playwright.options.HarNotFound;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.DisabledIf;
 import org.junit.jupiter.api.io.TempDir;
 
 import java.io.IOException;
@@ -213,6 +214,7 @@ void shouldGoBackToRedirectedNavigation() {
   }
 
   @Test
+  @DisabledIf(value="isFirefox", disabledReason="Flaky in Firefox, upstream as well")
   void shouldGoForwardToRedirectedNavigation() {
     Path path = Paths.get("src/test/resources/har-redirect.har");
     context.routeFromHAR(path, new BrowserContext.RouteFromHAROptions().setUrl(Pattern.compile(".*theverge.*")));

scripts/CLI_VERSION

@@ -1 +1 @@
-1.23.0-beta-1656035897000
+1.23.0