Summary: | [Model] clean up compile-time flags and condition IDL methods behind them | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Antoine Quint <graouts> | ||||||
Component: | New Bugs | Assignee: | Antoine Quint <graouts> | ||||||
Status: | REOPENED --- | ||||||||
Severity: | Normal | CC: | benjamin, cdumez, cmarcelo, commit-queue, dino, esprehn+autocc, ews-watchlist, kondapallykalyan, thorton, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | 233652 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Antoine Quint
2021-11-21 03:09:53 PST
Created attachment 444923 [details]
Patch
r- because it seems like this entirely regresses https://trac.webkit.org/changeset/282567/webkit; I'll retract my r- if you build with SEPARATED_MODEL on and everything works. Created attachment 445077 [details]
Patch
(In reply to Tim Horton from comment #2) > r- because it seems like this entirely regresses > https://trac.webkit.org/changeset/282567/webkit; I'll retract my r- if you > build with SEPARATED_MODEL on and everything works. The new patch retains ENABLE_ARKIT_INLINE_PREVIEW accounting for the changes made in r 282567. Comment on attachment 445077 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445077&action=review > Source/WebCore/PAL/pal/spi/cocoa/QuartzCoreSPI.h:58 > -#if ENABLE(ARKIT_INLINE_PREVIEW_IOS) > +#if HAVE(ASV_INLINE_PREVIEW_IOS) Should just remove this ifdef, it's a lie > Source/WebCore/PAL/pal/spi/cocoa/QuartzCoreSPI.h:95 > -#if ENABLE(ARKIT_INLINE_PREVIEW_IOS) > +#if HAVE(ASV_INLINE_PREVIEW_IOS) Should just remove this ifdef, it's a lie Committed r286227 (244608@main): <https://commits.webkit.org/244608@main> Re-opened since this is blocked by bug 233652 |