Bug 240378 – Non-unified build broken in debug mode

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Adrian Perez]

/app/webkit/Source/JavaScriptCore/heap/StructureAlignedMemoryAllocator.cpp: In static member function ‘static void JSC::StructureMemoryManager::commitBlock(void*)’:
/app/webkit/Source/JavaScriptCore/heap/StructureAlignedMemoryAllocator.cpp:123:49: error: ‘PROT_READ’ was not declared in this scope
  123 |         mprotect(block, MarkedBlock::blockSize, PROT_READ | PROT_WRITE);
      |                                                 ^~~~~~~~~
/app/webkit/Source/JavaScriptCore/heap/StructureAlignedMemoryAllocator.cpp:123:61: error: ‘PROT_WRITE’ was not declared in this scope
  123 |         mprotect(block, MarkedBlock::blockSize, PROT_READ | PROT_WRITE);
      |                                                             ^~~~~~~~~~
/app/webkit/Source/JavaScriptCore/heap/StructureAlignedMemoryAllocator.cpp:123:9: error: ‘mprotect’ was not declared in this scope
  123 |         mprotect(block, MarkedBlock::blockSize, PROT_READ | PROT_WRITE);
      |         ^~~~~~~~
/app/webkit/Source/JavaScriptCore/heap/StructureAlignedMemoryAllocator.cpp: In static member function ‘static void JSC::StructureMemoryManager::decommitBlock(void*)’:
/app/webkit/Source/JavaScriptCore/heap/StructureAlignedMemoryAllocator.cpp:135:49: error: ‘PROT_NONE’ was not declared in this scope; did you mean ‘U_NT_NONE’?
  135 |         mprotect(block, MarkedBlock::blockSize, PROT_NONE);
      |                                                 ^~~~~~~~~
      |                                                 U_NT_NONE
/app/webkit/Source/JavaScriptCore/heap/StructureAlignedMemoryAllocator.cpp:135:9: error: ‘mprotect’ was not declared in this scope
  135 |         mprotect(block, MarkedBlock::blockSize, PROT_NONE);
      |         ^~~~~~~~

[Adrian Perez]

We need to include <sys/mman.h> for OS(UNIX).

[Adrian Perez]

Some more build issues with build-webkit --gtk --debug:

/app/webkit/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp: In function ‘std::error_code WebCore::ContentExtensions::compileRuleList(WebCore::ContentExtensions::ContentExtensionCompilationClient&, WTF::String&&, WTF::Vector<WebCore::ContentExtensions::ContentExtensionRule>&&)’:
/app/webkit/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp:274:76: error: ‘crossThreadCopy’ was not declared in this scope
  274 |     callOnMainThread([ruleJSON = ruleJSON.isolatedCopy(), parsedRuleList = crossThreadCopy(parsedRuleList)] {
      |                                                                            ^~~~~~~~~~~~~~~
In file included from /app/webkit/WebKitBuild/Debug/WTF/Headers/wtf/StdLibExtras.h:33,
                 from /app/webkit/WebKitBuild/Debug/WTF/Headers/wtf/FastMalloc.h:26,
                 from /app/webkit/Source/WebCore/config.h:56,
                 from /app/webkit/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp:26:
/app/webkit/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp: In lambda function:
/app/webkit/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp:275:51: error: ‘parsedRuleList’ is not captured
  275 |         ASSERT(parseRuleList(ruleJSON).value() == parsedRuleList);
      |                                                   ^~~~~~~~~~~~~~

[Adrian Perez]

One more issue:

In file included from /app/webkit/WebKitBuild/Debug/WTF/Headers/wtf/StdLibExtras.h:33,
                 from /app/webkit/WebKitBuild/Debug/WTF/Headers/wtf/FastMalloc.h:26,                         
                 from /app/webkit/Source/WebCore/config.h:56,                                                                                                  
                 from /app/webkit/Source/WebCore/workers/service/ServiceWorkerClients.cpp:26:
/app/webkit/Source/WebCore/workers/service/ServiceWorkerClients.cpp: In member function ‘void WebCore::ServiceWorkerClients::openWindow(WebCore::ScriptExecutio
nContext&, const WTF::String&, WTF::Ref<WebCore::DeferredPromise>&&)’:
/app/webkit/WebKitBuild/Debug/WTF/Headers/wtf/Assertions.h:174:56: error: ‘LOG_CHANNEL_PREFIXServiceWorker’ was not declared in this scope
  174 | #define LOG_CHANNEL(name) JOIN_LOG_CHANNEL_WITH_PREFIX(LOG_CHANNEL_PREFIX, name)
      |                                                        ^~~~~~~~~~~~~~~~~~

[Adrian Perez]

Pull request: https://github.com/WebKit/WebKit/pull/608

[EWS]

Committed r294177 (250544@main): <https://commits.webkit.org/250544@main>

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

[Radar WebKit Bug Importer]

<rdar://problem/93269326>