Revert "feat(evaluate): allow passing functions as evaluate arguments (#41801)"

[pavelfeldman]

No description provided.

[github-actions]

Test results for "MCP"

1 failed
❌ [chrome] › mcp/annotate.spec.ts:110 › should abort annotation when last screenshot is removed @mcp-macos-latest-chrome

7759 passed, 1249 skipped

---

Merge workflow run.

[github-actions]

Test results for "tests 1"

7 flaky

⚠️ [chromium-library] › library/video.spec.ts:736 › screencast › should work with video+trace `@frozen-time-library-chromium-linux`
⚠️ [chromium-library] › library/chromium/oopif.spec.ts:282 › should click `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:736 › screencast › should work with video+trace `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/beforeunload.spec.ts:130 › should support dismissing the dialog multiple times `@realtime-time-library-chromium-linux`
⚠️ [chromium-library] › library/chromium/chromium.spec.ts:371 › should produce network events, routing, and annotations for Service Worker `@chromium-ubuntu-22.04-node20`
⚠️ [chromium-library] › library/popup.spec.ts:260 › should not throw when click closes popup `@chromium-ubuntu-22.04-node20`
⚠️ [chromium-library] › library/video.spec.ts:736 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node20`

49802 passed, 1188 skipped

---

Merge workflow run.

[github-actions]

Hi, I'm the Playwright bot and I took a first look at the CI failures.

🟢 The one failure is a pre-existing flake — the PR is clear

[chrome] › mcp/annotate.spec.ts:110 › should abort annotation when last screenshot is removed fails on main and many unrelated PRs, and this revert doesn't touch the MCP annotate flow.

Details

Only one real failure in the report; everything else passed. This PR reverts the "functions as evaluate arguments" feature — it touches evaluate serialization (utilityScriptSerializers, bindingsController, page evaluate, ct-core serializers). None of that is on the MCP screenshot/annotate code path.

Pre-existing flake / infra

• [chrome] › mcp/annotate.spec.ts:110 › should abort annotation when last screenshot is removed — established flake across the test-results DB. It flips verdict on SHAs this PR can't be responsible for: main (e.g. sha 56e57f2c, 0cbee5ab, d333a3ac) and unrelated PRs like feat(webkit): roll to r2334 #41879 (webkit roll), feat(aria): treat frame elements like iframes in aria snapshots #41828 (frameset-aria-snapshot), chore(test-runner): address httpCache review comments #41835 (http-cache-proxy). Failure counts per project: firefox 21/671, chromium 12/682, chrome 12/682, webkit 5/682 — i.e. it fails on every browser, ~2–3% of runs, with no tie to the evaluate code this PR reverts.

_{Triaged by the Playwright bot - agent run}