fix: correctly encode unicode strings to UTF-8 (#167)

Author: yury-s

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

@@ -37,8 +37,7 @@ public class Transport {
     DataInputStream in = new DataInputStream(new BufferedInputStream(input));
     readerThread = new ReaderThread(in, incoming);
     readerThread.start();
-    DataOutputStream out = new DataOutputStream(output);
-    writerThread = new WriterThread(out, outgoing);
+    writerThread = new WriterThread(output, outgoing);
     writerThread.start();
   }
 
@@ -125,17 +124,17 @@ private String readMessage() throws IOException {
 }
 
 class WriterThread extends Thread {
-  final DataOutputStream out;
+  final OutputStream out;
   private final BlockingQueue<String> queue;
 
-  private static void writeIntLE(DataOutputStream out, int v) throws IOException {
+  private static void writeIntLE(OutputStream out, int v) throws IOException {
     out.write(v >>> 0 & 255);
     out.write(v >>> 8 & 255);
     out.write(v >>> 16 & 255);
     out.write(v >>> 24 & 255);
   }
 
-  WriterThread(DataOutputStream out, BlockingQueue<String> queue) {
+  WriterThread(OutputStream out, BlockingQueue<String> queue) {
     this.out = out;
     this.queue = queue;
   }
@@ -158,8 +157,8 @@ public void run() {
   }
 
   private void sendMessage(String message) throws IOException {
-    int len = message.length();
-    writeIntLE(out, len);
-    out.writeBytes(message);
+    byte[] bytes = message.getBytes(StandardCharsets.UTF_8);
+    writeIntLE(out, bytes.length);
+    out.write(bytes);
   }
 }

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

@@ -511,6 +511,13 @@ void shouldClimbDomForInnerLabelWithPointerEventsNone() {
     assertEquals(true, page.evaluate("__CLICKED"));
   }
 
+  @Test
+  void shouldWorkWithUnicodeSelectors() {
+    page.setContent("<button onclick='javascript:window.__CLICKED=true;'><label style='pointer-events:none'>Найти</label></button>");
+    page.click("text=Найти");
+    assertEquals(true, page.evaluate("__CLICKED"));
+  }
+
   @Test
   void shouldClimbUpTo_roleButton_() {
     page.setContent("<div role=button onclick='javascript:window.__CLICKED=true;'><div style='pointer-events:none'><span><div>Click target</div></span></div>");