Bug 232519 introduces some changes to unify the memoryPressureMonitorDisabled implementation. It does this by calling a new static method `MemoryPressureMonitor::disabled`. It appears the MemoryPressureMonitor class is conditionally defined on OS(LINUX). For non-Linux GTK builds, I am getting: /build/rtutils/components/desktop/webkitgtk4-dev/webkit/Source/WebKit/UIProcess/glib/WebProcessPoolGLib.cpp:97:9: error: 'MemoryPressureMonitor' has not been declared 97 | if (MemoryPressureMonitor::disabled()) | ^~~~~~~~~~~~~~~~~~~~~ /build/rtutils/components/desktop/webkitgtk4-dev/webkit/Source/WebKit/UIProcess/soup/WebProcessPoolSoup.cpp:46:9: error: 'MemoryPressureMonitor' has not been declared 46 | if (MemoryPressureMonitor::disabled()) | ^~~~~~~~~~~~~~~~~~~~~
Created attachment 443307 [details] Build fix
Committed r285265 (243876@main): <https://commits.webkit.org/243876@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 443307 [details].
Thanks for the quick fix! You missed one: /build/rtutils/components/desktop/webkitgtk4-dev/webkit/Source/WebKit/UIProcess/soup/WebProcessPoolSoup.cpp:46:9: error: 'MemoryPressureMonitor' has not been declared 46 | if (MemoryPressureMonitor::disabled()) | ^~~~~~~~~~~~~~~~~~~~~
Created attachment 443603 [details] Patch
Committed r285435 (243991@main): <https://commits.webkit.org/243991@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 443603 [details].
I can confirm, it now builds as expected. Thanks!
(In reply to Jim Mason from comment #6) > I can confirm, it now builds as expected. Thanks! You're welcome! 👋️