fix: Restore runtime compatibility with System.Text.Json 6.0 (#2904)

* Restore runtime compatibility with System.Text.Json 6.0, which doesn't have the ValueIsEscaped property.

* ignore flaky test

---------

Co-authored-by: Darío Kondratiuk <[email protected]>

Author: leonardo-fernandes

lib/PuppeteerSharp.Tests/WorkerTests/PageWorkerTests.cs

@@ -12,6 +12,7 @@ public PageWorkerTests() : base()
         }
 
         [Test, Retry(2), PuppeteerTest("worker.spec", "Workers", "Page.workers")]
+        [Ignore("TODO: Fix me. Too flaky")]
         public async Task PageWorkers()
         {
             var workerCreatedTcs = new TaskCompletionSource<bool>();

lib/PuppeteerSharp/Helpers/Json/LowSurrogateConverter.cs

@@ -26,12 +26,7 @@ public override string Read(ref Utf8JsonReader reader, Type typeToConvert, JsonS
 
         try
         {
-            if (reader.ValueIsEscaped)
-            {
-                value = JsonUnescape(value);
-            }
-
-            return value;
+            return JsonUnescapedValue(reader, value);
         }
         catch
         {
@@ -44,9 +39,16 @@ public override void Write(Utf8JsonWriter writer, string value, JsonSerializerOp
         writer.WriteStringValue(value);
     }
 
-    private static string JsonUnescape(string jsonString)
+    private static string JsonUnescapedValue(Utf8JsonReader reader, string jsonString)
     {
-        using var doc = JsonDocument.Parse($"\"{jsonString}\"");
-        return doc.RootElement.GetString();
+        if (reader.ValueIsEscaped)
+        {
+            using var doc = JsonDocument.Parse($"\"{jsonString}\"");
+            return doc.RootElement.GetString();
+        }
+        else
+        {
+            return jsonString;
+        }
     }
 }