Summary: | [WPE][WebDriver] Implement automation keyboard and mouse input handlers for new WPE API | ||
---|---|---|---|
Product: | WebKit | Reporter: | Lauro Moura <lmoura> |
Component: | WPE WebKit | Assignee: | Diego Pino <dpino> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | bugs-noreply, dpino |
Priority: | P2 | ||
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=275423 | ||
Bug Depends on: | |||
Bug Blocks: | 265646, 274776 |
Description
Lauro Moura
2024-05-05 18:46:23 PDT
Pull request: https://github.com/WebKit/WebKit/pull/28155 Committed 279376@main (4a97b2ece9b5): <https://commits.webkit.org/279376@main> Reviewed commits have been landed. Closing PR #28155 and removing active labels. WPE Debug build bot is failing with error: https://build.webkit.org/#/builders/66/builds/32268/steps/10/logs/stdio ``` In file included from /app/webkit/WebKitBuild/WPE/Debug/WTF/Headers/wtf/StdLibExtras.h:35, from /app/webkit/WebKitBuild/WPE/Debug/WTF/Headers/wtf/FastMalloc.h:26, from /app/webkit/Source/WebKit/config.h:42, from /app/webkit/Source/WebKit/UIProcess/Automation/libwpe/WebAutomationSessionWPE.cpp:26, from /app/webkit/WebKitBuild/WPE/Debug/DerivedSources/WebKit/unified-sources/UnifiedSource-88d1702b-23.cpp:1: /app/webkit/Source/WebKit/UIProcess/Automation/libwpe/WebAutomationSessionWPE.cpp: In function ‘void WebKit::doKeyStrokeEvent(WebPageProxy&, bool, uint32_t, uint32_t, bool)’: /app/webkit/WebKitBuild/WPE/Debug/WTF/Headers/wtf/Assertions.h:162:56: error: ‘LOG_CHANNEL_PREFIXAutomation’ was not declared in this scope; did you mean ‘LOG_CHANNEL_PREFIX’? 162 | #define LOG_CHANNEL(name) JOIN_LOG_CHANNEL_WITH_PREFIX(LOG_CHANNEL_PREFIX, name) | ^~~~~~~~~~~~~~~~~~ ``` Re-opening for pull request https://github.com/webkit/webkit/pull/29217 In case unified sources groups files 'UIProcess/Automation/libwpe/WebAutomationSessionLibWPE.cpp' and 'UIProcess/Automation/libwpe/WebAutomationSessionWPE.cpp', a duplicate symbol build error will happen: ``` Source/WebKit/CMakeFiles/WebKit.dir/__/__/DerivedSources/WebKit/unified-sources/UnifiedSource-88d1702b-23.cpp.o.d -o Source/WebKit/CMakeFiles/WebKit.dir/__/__/DerivedSources/WebKit/unified-sources/UnifiedSource-88d1702b-23.cpp.o -c DerivedSources/WebKit/unified-sources/UnifiedSource-88d1702b-23.cpp In file included from DerivedSources/WebKit/unified-sources/UnifiedSource-88d1702b-23.cpp:8: ../../../Source/WebKit/UIProcess/Automation/libwpe/WebAutomationSessionWPE.cpp:84:17: error: redefinition of 'modifiersToEventState' 84 | static uint32_t modifiersToEventState(OptionSet<WebEventModifier> modifiers) | ^ ../../../Source/WebKit/UIProcess/Automation/libwpe/WebAutomationSessionLibWPE.cpp:38:17: note: previous definition is here 38 | static uint32_t modifiersToEventState(OptionSet<WebEventModifier> modifiers) | ^ In file included from DerivedSources/WebKit/unified-sources/UnifiedSource-88d1702b-23.cpp:8: ../../../Source/WebKit/UIProcess/Automation/libwpe/WebAutomationSessionWPE.cpp:100:17: error: redefinition of 'mouseButtonToWPEButton' 100 | static unsigned mouseButtonToWPEButton(MouseButton button) | ^ ../../../Source/WebKit/UIProcess/Automation/libwpe/WebAutomationSessionLibWPE.cpp:50:17: note: previous definition is here 50 | static unsigned mouseButtonToWPEButton(MouseButton button) | ^ In file included from DerivedSources/WebKit/unified-sources/UnifiedSource-88d1702b-23.cpp:8: ../../../Source/WebKit/UIProcess/Automation/libwpe/WebAutomationSessionWPE.cpp:114:17: error: redefinition of 'stateModifierForWPEButton' 114 | static unsigned stateModifierForWPEButton(unsigned button) | ^ ../../../Source/WebKit/UIProcess/Automation/libwpe/WebAutomationSessionLibWPE.cpp:64:17: note: previous definition is here 64 | static unsigned stateModifierForWPEButton(unsigned button) | ^ ``` Pull request: https://github.com/webkit/webkit/pull/29306 Committed 279538@main (5ca7f83342d3): <https://commits.webkit.org/279538@main> Reviewed commits have been landed. Closing PR #29306 and removing active labels. |