Bug 323694
| Summary: | REGRESSION(320518@main): C++ node resolution reports "no such element" for stale elements | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Lauro Moura <lmoura> |
| Component: | WebDriver | Assignee: | Brent Fulgham <bfulgham> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bburg, bfulgham, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=323692 https://bugs.webkit.org/show_bug.cgi?id=323695 https://bugs.webkit.org/show_bug.cgi?id=323386 |
||
Lauro Moura
320518@main added a new internal Automation error type, "StaleNode". This can be raised by AutomationSessionProxy.js::_nodeForIdentifier, but the wrapper nodeForIdentifier swallows all errors, returning only null. This in turn surfaces as a "no such element" instead of the expected "stale element reference".
This is causing regressions in the following tests:
imported/w3c/webdriver/tests/classic/get_computed_label/get.py::test_stale_element_reference[top_context]
imported/w3c/webdriver/tests/classic/get_computed_label/get.py::test_stale_element_reference[child_context]
imported/w3c/webdriver/tests/classic/get_computed_role/get.py::test_stale_element_reference[top_context]
imported/w3c/webdriver/tests/classic/get_computed_role/get.py::test_stale_element_reference[child_context]
imported/w3c/webdriver/tests/classic/get_element_rect/get.py::test_stale_element_reference[top_context]
imported/w3c/webdriver/tests/classic/get_element_rect/get.py::test_stale_element_reference[child_context]
imported/w3c/webdriver/tests/classic/perform_actions/pointer_mouse.py::test_stale_element_reference[top_context]
imported/w3c/webdriver/tests/classic/perform_actions/pointer_mouse.py::test_stale_element_reference[child_context]
imported/w3c/webdriver/tests/classic/perform_actions/pointer_pen.py::test_stale_element_reference[top_context]
imported/w3c/webdriver/tests/classic/perform_actions/pointer_pen.py::test_stale_element_reference[child_context]
imported/w3c/webdriver/tests/classic/perform_actions/pointer_touch.py::test_stale_element_reference[top_context]
imported/w3c/webdriver/tests/classic/perform_actions/pointer_touch.py::test_stale_element_reference[child_context]
imported/w3c/webdriver/tests/classic/switch_to_frame/switch_webelement.py::test_frame_id_webelement_stale_element_reference[top_context]
imported/w3c/webdriver/tests/classic/switch_to_frame/switch_webelement.py::test_frame_id_webelement_stale_element_reference[child_context]
imported/w3c/webdriver/tests/classic/take_element_screenshot/screenshot.py::test_stale_element_reference[top_context]
imported/w3c/webdriver/tests/classic/take_element_screenshot/screenshot.py::test_stale_element_reference[child_context]
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/187509909>
Brent Fulgham
*** This bug has been marked as a duplicate of bug 323692 ***
Brent Fulgham
I'm reopening to land the corrected test cases, though.
Brent Fulgham
Pull request: https://github.com/WebKit/WebKit/pull/74862
EWS
Test gardening commit 321858@main (4b3249f2dd32): <https://commits.webkit.org/321858@main>
Reviewed commits have been landed. Closing PR #74862 and removing active labels.