fix(evaluate): make sure to try/catch toJSON access (#27238)

packages/playwright-core/src/server/isomorphic/utilityScriptSerializers.ts

@@ -208,9 +208,15 @@ export function source() {
           o.push({ k: name, v: serialize(item, handleSerializer, visitorInfo) });
       }
 
-      // If Object.keys().length === 0 we fall back to toJSON if it exists
+      let jsonWrapper;
-      if (o.length === 0 && value.toJSON && typeof value.toJSON === 'function')
+      try {
-        return innerSerialize(value.toJSON(), handleSerializer, visitorInfo);
+        // If Object.keys().length === 0 we fall back to toJSON if it exists
+        if (o.length === 0 && value.toJSON && typeof value.toJSON === 'function')
+          jsonWrapper = { value: value.toJSON() };
+      } catch (e) {
+      }
+      if (jsonWrapper)
+        return innerSerialize(jsonWrapper.value, handleSerializer, visitorInfo);
 
       return { o, id };
     }

tests/page/page-evaluate.spec.ts

@@ -654,6 +654,23 @@ it('should not use toJSON in jsonValue', async ({ page }) => {
   expect(await resultHandle.jsonValue()).toEqual({ data: 'data', toJSON: {} });
 });
 
+it('should ignore buggy toJSON', async ({ page }) => {
+  const result = await page.evaluate(() => {
+    class Foo {
+      toJSON() {
+        throw new Error('Bad');
+      }
+    }
+    class Bar {
+      get toJSON() {
+        throw new Error('Also bad');
+      }
+    }
+    return { foo: new Foo(), bar: new Bar() };
+  });
+  expect(result).toEqual({ foo: {}, bar: {} });
+});
+
 it('should not expose the injected script export', async ({ page }) => {
   expect(await page.evaluate('typeof pwExport === "undefined"')).toBe(true);
 });