Skip to content

REGRESSION(291284@main) WebAutomationSession needs to push a command failure as soon as it gets it#42011

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
lauromoura:eng/REGRESSION-291284-main-WebAutomationSession-needs-to-push-a-command-failure-as-soon-as-it-gets-it
Mar 13, 2025
Merged

REGRESSION(291284@main) WebAutomationSession needs to push a command failure as soon as it gets it#42011
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
lauromoura:eng/REGRESSION-291284-main-WebAutomationSession-needs-to-push-a-command-failure-as-soon-as-it-gets-it

Conversation

@lauromoura
Copy link
Copy Markdown
Contributor

@lauromoura lauromoura commented Mar 6, 2025

cc06097

REGRESSION(291284@main) WebAutomationSession needs to push a command failure as soon as it gets it
https://bugs.webkit.org/show_bug.cgi?id=289259

Reviewed by BJ Burg.

Ensure we flush the newly reported error as soon as we get them, to
avoid clients hanging waiting for a reply.

This aligns with the original CallbackBase::sendFailure behavior when
an error was reported.

Also make sure we report the right request id in the error.

* Source/JavaScriptCore/inspector/scripts/codegen/generate_cpp_backend_dispatcher_implementation.py:
(CppBackendDispatcherImplementationGenerator._generate_dispatcher_implementation_for_command):
* Source/JavaScriptCore/inspector/scripts/tests/expected/commands-with-async-attribute.json-result:

Canonical link: https://commits.webkit.org/292050@main

86594c4

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 win
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 ✅ 🧪 win-tests
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt ✅ 🧪 mac-wk1 ✅ 🛠 wpe-cairo
✅ 🛠 🧪 jsc 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🛠 gtk
✅ 🛠 🧪 jsc-arm64 ✅ 🛠 vision 🧪 mac-AS-debug-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🧪 api-gtk
✅ 🛠 🧪 merge ✅ 🧪 vision-wk2 🧪 mac-intel-wk2 ✅ 🛠 playstation
✅ 🛠 tv ✅ 🛠 mac-safer-cpp ✅ 🛠 jsc-armv7
✅ 🛠 tv-sim 🧪 jsc-armv7-tests
✅ 🛠 watch
✅ 🛠 watch-sim

@lauromoura lauromoura self-assigned this Mar 6, 2025
@lauromoura lauromoura added the WebDriver Bugs in WebDriver label Mar 6, 2025
@lauromoura lauromoura requested a review from burg March 6, 2025 17:51
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Mar 6, 2025
@lauromoura lauromoura removed the merging-blocked Applied to prevent a change from being merged label Mar 7, 2025
Copy link
Copy Markdown
Contributor

@burg burg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please include rebaselined test results after this PR lands:
#42322

@lauromoura lauromoura force-pushed the eng/REGRESSION-291284-main-WebAutomationSession-needs-to-push-a-command-failure-as-soon-as-it-gets-it branch from e3a081b to 86594c4 Compare March 12, 2025 21:40
@burg burg added the merge-queue Applied to send a pull request to merge-queue label Mar 12, 2025
…failure as soon as it gets it

https://bugs.webkit.org/show_bug.cgi?id=289259

Reviewed by BJ Burg.

Ensure we flush the newly reported error as soon as we get them, to
avoid clients hanging waiting for a reply.

This aligns with the original CallbackBase::sendFailure behavior when
an error was reported.

Also make sure we report the right request id in the error.

* Source/JavaScriptCore/inspector/scripts/codegen/generate_cpp_backend_dispatcher_implementation.py:
(CppBackendDispatcherImplementationGenerator._generate_dispatcher_implementation_for_command):
* Source/JavaScriptCore/inspector/scripts/tests/expected/commands-with-async-attribute.json-result:

Canonical link: https://commits.webkit.org/292050@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/REGRESSION-291284-main-WebAutomationSession-needs-to-push-a-command-failure-as-soon-as-it-gets-it branch from 86594c4 to cc06097 Compare March 13, 2025 01:17
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 292050@main (cc06097): https://commits.webkit.org/292050@main

Reviewed commits have been landed. Closing PR #42011 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit cc06097 into WebKit:main Mar 13, 2025
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Mar 13, 2025
@lauromoura lauromoura deleted the eng/REGRESSION-291284-main-WebAutomationSession-needs-to-push-a-command-failure-as-soon-as-it-gets-it branch September 8, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WebDriver Bugs in WebDriver

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants