roll to 1.53.0

Author: Skn0tt

README.md

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

playwright/src/main/java/com/microsoft/playwright/impl/SharedSelectors.java

@@ -72,13 +72,13 @@ private void registerImpl(String name, String script, RegisterOptions options) {
     JsonObject engine = new JsonObject();
     engine.addProperty("name", name);
     engine.addProperty("source", script);
-    if (options.contentScript != null) {
+    if (options != null && options.contentScript != null) {
       engine.addProperty("contentScript", options.contentScript);
     }
     for (BrowserContextImpl context : contextsForSelectors) {
       JsonObject params = new JsonObject();
       params.add("selectorEngine", engine);
-      context.sendMessageAsync("registerSelectorEngine", params);
+      context.sendMessage("registerSelectorEngine", params);
     }
     selectorEngines.add(engine);
   }

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

@@ -20,7 +20,6 @@
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.Assumptions;
 import org.junit.jupiter.api.BeforeAll;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
 
@@ -297,7 +296,6 @@ void shouldEmitCloseEventsOnPagesAndContexts() throws InterruptedException {
     assertEquals(Arrays.asList("page", "context"), events);
   }
 
-  @Disabled("Temporarily skipping until the roll that contains https://github.com/microsoft/playwright/pull/36227")
   @Test
   void shouldRespectSelectors() {
     String mycss = "{\n" +

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

@@ -20,7 +20,6 @@
 import com.microsoft.playwright.options.Geolocation;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Assumptions;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.condition.DisabledIf;
 import org.junit.jupiter.api.io.TempDir;
@@ -227,7 +226,6 @@ void coverageShouldBeMissing() {
     // TODO:
   }
 
-  @Disabled("Temporarily skipping until the roll that contains https://github.com/microsoft/playwright/pull/36227")
   @Test
   void shouldRespectSelectors() {
     Page page = launchPersistent();

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

@@ -16,7 +16,6 @@
 
 package com.microsoft.playwright;
 
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 import java.util.Collections;
@@ -113,7 +112,6 @@ void textContentShouldWork() {
     assertEquals("Text,\nmore text", page.textContent("#inner"));
   }
 
-  @Disabled("Temporarily skipping until the roll that contains https://github.com/microsoft/playwright/pull/36227")
   @Test
   void textContentShouldBeAtomic() {
     String createDummySelector = "{\n" +
@@ -137,7 +135,6 @@ void textContentShouldBeAtomic() {
     assertEquals("modified", page.evaluate("() => document.querySelector('div').textContent"));
   }
 
-  @Disabled("Temporarily skipping until the roll that contains https://github.com/microsoft/playwright/pull/36227")
   @Test
   void innerTextShouldBeAtomic() {
     String createDummySelector = "{\n" +
@@ -161,7 +158,6 @@ void innerTextShouldBeAtomic() {
     assertEquals("modified", page.evaluate("() => document.querySelector('div').innerText"));
   }
 
-  @Disabled("Temporarily skipping until the roll that contains https://github.com/microsoft/playwright/pull/36227")
   @Test
   void innerHTMLShouldBeAtomic() {
     String createDummySelector = "{\n" +
@@ -185,7 +181,6 @@ void innerHTMLShouldBeAtomic() {
     assertEquals("modified", page.evaluate("() => document.querySelector('div').innerHTML"));
   }
 
-  @Disabled("Temporarily skipping until the roll that contains https://github.com/microsoft/playwright/pull/36227")
   @Test
   void getAttributeShouldBeAtomic() {
     String createDummySelector = "{\n" +

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

@@ -16,14 +16,12 @@
 
 package com.microsoft.playwright;
 
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 import java.nio.file.Paths;
 
 import static org.junit.jupiter.api.Assertions.*;
 
-@Disabled("Temporarily skipping until the roll that contains https://github.com/microsoft/playwright/pull/36227")
 public class TestSelectorsRegister extends TestBase {
   @Test
   void shouldWork() {

scripts/DRIVER_VERSION

@@ -1 +1 @@
-1.53.0-beta-1749131401000
+1.53.0