Skip to content

fix(dom): validate XPathResult receivers and preserve native alias state - #1864

Open
ldm0 wants to merge 1 commit into
mainfrom
wpt/20261022-xpath-result-native-receivers
Open

ldm0 wants to merge 1 commit into
mainfrom
wpt/20261022-xpath-result-native-receivers

Conversation

@ldm0

@ldm0 ldm0 commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

XPathResult methods and accessors now use generated native receiver validation and resolve registered native Proxy aliases before accessing result state. Keep result-type checks in the XPath implementation while centralizing interface identity at the binding boundary.

Regression coverage checks descriptors, per-result behavior, cross-realm errors and shared native alias state.

Validation on base a30f889caa12: cargo fmt --all; cargo clippy --workspace --all-targets --all-features -- -D warnings; cargo nextest run --no-fail-fast. All passed (20,530 passed, 18 existing skips; 32 build jobs and test threads, no automatic retries). Focused regressions also passed.

@github-actions

Copy link
Copy Markdown

Sequential Navigation Soak A/B

⚠️ Benchmark infrastructure failed before a comparison could be produced.

Workflow: timed_out. Inspect the workflow run for the failing build, harness, or artifact step.

No navigation resilience or memory conclusion is available for this run.

@github-actions

Copy link
Copy Markdown

Spider Bench A/B

⚠️ Spider Bench infrastructure failed before a comparison could be produced.

Workflow: timed_out. Inspect the workflow run for the failing build, harness, or artifact step.

No benchmark result or performance conclusion is available for this run.

@github-actions

Copy link
Copy Markdown

CI Regression Report

Source CI run · source state at render: timed_out · artifacts: 2/6

Check Status Signal
Release regression ⚪ artifact unavailable or invalid
Frontend differential ⚪ artifact unavailable or invalid
Agent episodes ⚪ artifact unavailable or invalid
Runtime/CDP contracts ⚪ artifact unavailable or invalid
CDP smoke ✅ 52/52 groups passed; 622 scenarios
WebMainBench ✅ 545/545 completed; 0 unexpected failures
Release regression — ⚪ artifact unavailable or invalid

Artifact unavailable or invalid. See the source CI run for infrastructure details.

Frontend differential — ⚪ artifact unavailable or invalid

Artifact unavailable or invalid. See the source CI run for infrastructure details.

Agent episodes · Moli vs Chromium — ⚪ artifact unavailable or invalid

Artifact unavailable or invalid. See the source CI run for infrastructure details.

Runtime and CDP session contracts — ⚪ artifact unavailable or invalid

Artifact unavailable or invalid. See the source CI run for infrastructure details.

CDP smoke — ✅ 52/52 groups passed; 622 scenarios

Workers: 4 · cumulative group time: 140.88 s · failed groups: 0

Slowest group Scenarios Duration Status
scrollbar-visibility 22 14.38 s ✅
target-lifecycle 24 11.26 s ✅
dom-input 27 9.29 s ✅
chromium-cdp 56 8.96 s ✅
multi-page 45 8.23 s ✅
WebMainBench · 545 pages — ✅ 545/545 completed; 0 unexpected failures
Completed Successful Expected DNS failure Unexpected failures Panics Crashes Timeouts Empty output
545/545 544 1 0 0 0 0 0

Frozen HTML · Linux · --wait done · 45 seconds per page · no retries. Content quality is evaluated separately.

Diagnostics: webmainbench-results — every page's Markdown, stderr, and exit status. Source run and artifacts

All artifact fields are parsed by the trusted default-branch renderer; missing or invalid inputs remain visible as unavailable.

@ldm0
ldm0 force-pushed the wpt/20261022-xpath-result-native-receivers branch from 6f33b7a to 788d053 Compare October 22, 2026 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant