fix(aria): process children of hidden elements

[Skn0tt]

Closes #36296

[Copilot]

Pull Request Overview

This PR addresses processing of hidden elements for ARIA snapshots in response to issue #36296. Key changes include:

• Adding a new test to verify that visible children of hidden elements are correctly captured.
• Introducing the helper function isVisible in the ARIA snapshot generation, and modifying text node and element handling accordingly.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
tests/page/page-aria-snapshot.spec.ts	Adds a test verifying that visible children within hidden elements are processed.
packages/injected/src/ariaSnapshot.ts	Introduces a new isVisible helper function and updates element processing logic to incorporate it.

Comments suppressed due to low confidence (2)

packages/injected/src/ariaSnapshot.ts:50

• Consider adding a brief comment above the isVisible function to explain its purpose in determining element visibility for ARIA processing.

function isVisible(element: Element, options?: { forAI?: boolean }): boolean {

tests/page/page-aria-snapshot.spec.ts:662

• [nitpick] For consistency with other test names, consider renaming the test title to begin with 'should', for example: "should show visible children of hidden elements".

it('show visible children of hidden elements', { annotation: { type: 'issue', description: 'https://github.com/microsoft/playwright/issues/36296' }  }, async ({ page }) => {

[github-actions]

Test results for "tests 1"

1 failed
❌ [webkit-library] › library/browsercontext-pages.spec.ts:105:3 › should return bounding box with page scale @webkit-ubuntu-22.04-node18

2 flaky

⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:986:7 › cli codegen › should not throw csp directive violation errors @firefox-ubuntu-22.04-node18
⚠️ [webkit-page] › page/page-screenshot.spec.ts:345:5 › page screenshot › should work while navigating @webkit-ubuntu-22.04-node18

39384 passed, 822 skipped
✔️✔️✔️

Merge workflow run.