Bug 309353
| Summary: | [ATSPI] Add missing RenderElementInlines.h to AccessibilityObjectAtspi.cpp | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Lauro Moura <lmoura> |
| Component: | Accessibility | Assignee: | Lauro Moura <lmoura> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | andresg_22, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Lauro Moura
While testing PGO and LTO in the WPE build with clang, we had to use `--allow-multiple-definition`, which in turn made linking fail with the following error:
FAILED: lib/libWPEWebKit-2.0.so.1.10.0
: && /usr/local/bin/clang++ -fPIC -DASSERT_ENABLED=0 -flto=thin -fdiagnostics-color=always -fcolor-diagnostics -Wextra -Wall -Werror=undefined-internal -Werror=undefined-inline -pipe -Wno-noexcept-type -frelaxed-template-template-args -Wno-nullability-completeness -Wno-psabi -Wno-misleading-indentation -Wno-parentheses-equality -Qunused-arguments -Wundef -Wpointer-arith -Wmissing-format-attribute -Wformat-security -Wcast-align -Wno-tautological-compare -fasynchronous-unwind-tables -fdebug-types-section -fno-omit-frame-pointer -fno-strict-aliasing -fno-exceptions -fno-rtti -fcoroutines -ffunction-sections -fdata-sections -O3 -DNDEBUG -fprofile-generate=/tmp/pgo-profiles -Wl,--allow-multiple-definition -Wl,--no-undefined -flto=thin -L/jhbuild/install/lib -fuse-ld=lld -Wl,--gc-sections -Wl,--disable-new-dtags -shared -Wl,-soname,libWPEWebKit-2.0.so.1 -o lib/libWPEWebKit-2.0.so.1.10.0 @CMakeFiles/WebKit.rsp && :
ld.lld: error: undefined symbol: WebCore::RenderElement::isAnonymousBlock() const
>>> referenced by UnifiedSource-aba958d6-6.cpp
>>> Source/WebCore/CMakeFiles/WebCore.dir/__/__/WebCore/DerivedSources/unified-sources/libWPEWebKit-2.0.so.1.10.0.lto.UnifiedSource-aba958d6-6.cpp.o:(WebCore::AccessibilityObject::accessibilityPlatformIncludesObject() const)
Upon closer inspection, AccessibilityObjectAtspi.cpp indeed uses the inline defined in RenderElementInlines.h, without including it directly.
Patch incoming.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/171907503>
Lauro Moura
Pull request: https://github.com/WebKit/WebKit/pull/60069
EWS
Committed 308810@main (5948fe823902): <https://commits.webkit.org/308810@main>
Reviewed commits have been landed. Closing PR #60069 and removing active labels.