[py] expected_conditions: correct type annotation

[Delta456]

User description

Thanks for contributing to Selenium!
A PR well described will help maintainers to quickly review and merge it

Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, help reviewers by making them as simple and short as possible.

Motivation and Context

Fixes #15334

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• I have read the contributing document.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

---

PR Type

Bug fix, Documentation

---

Description

• Updated type annotation for frame_to_be_available_and_switch_to_it function to include WebElement.

• Improved docstring to reflect updated type annotation and clarify parameter usage.

• Ensured alignment of type annotations with function implementation and behavior.

---

Changes walkthrough 📝

	Relevant files
Bug fix	expected_conditions.py Update type annotation and docstring for frame_to_be_available_and_switch_to_it py/selenium/webdriver/support/expected_conditions.py Updated type annotation for locator parameter to include WebElement. Modified docstring to reflect the updated type annotation. +2/-2

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅ 15334 - PR Code Verified Compliant requirements: Update type annotation for frame_to_be_available_and_switch_to_it function to include WebElement type Update function's docstring to reflect the new type annotation Requires further human verification: Ensure type annotation matches actual function implementation (need to verify the function's implementation accepts WebElement)

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Missing Tests No tests were added to verify that the function works correctly with WebElement input, which is important for type annotation changes def frame_to_be_available_and_switch_to_it(locator: Union[Tuple[str, str], str, WebElement]) -> Callable[[WebDriver], bool]:

[qodo-merge-pro]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Handle stale WebElement case Add validation to handle case when WebElement is stale before attempting frame switch, to prevent potential runtime errors. py/selenium/webdriver/support/expected_conditions.py [488] def frame_to_be_available_and_switch_to_it(locator: Union[Tuple[str, str], str, WebElement]) -> Callable[[WebDriver], bool]: + if isinstance(locator, WebElement): + try: + locator.is_enabled() # Basic check if element is stale + except StaleElementReferenceException: + return lambda _: False Apply this suggestion Suggestion importance[1-10]: 8 __ Why: The suggestion addresses a critical edge case where a stale WebElement could cause runtime errors, by adding proper validation before attempting to switch frames. This is particularly important for WebDriver operations to maintain stability and prevent crashes.	Medium
Update

[cgoldberg]

@Delta456 doh! you beat me by 5 minutes :)

I'm closing my PR and will review yours.

[cgoldberg]

see updated change request

[cgoldberg]

FWIW, the linter is failing on this... you can break your change into 2 lines to fix it.

see: https://github.com/SeleniumHQ/selenium/actions/runs/13528688990/job/37805902044?pr=15337

[cgoldberg]

break into 2 lines to satisfy linter

[cgoldberg]

LGTM 👍