Bug 236129 – Fix for non-unified builds after r286821 (245056@main)

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Pablo Saavedra]

SSIA


Error:

In file included from /home/bot/workspace/sources/wpewebkit/Source/WebCore/workers/shared/SharedWorkerScriptLoader.cpp:27:
/home/bot/workspace/sources/wpewebkit/Source/WebCore/workers/shared/SharedWorkerScriptLoader.h:51:5: error: 'SharedWorkerScriptLoaderIdentifier' does not name a type; did you mean 'SharedWorkerScriptLoader'?
   51 |     SharedWorkerScriptLoaderIdentifier identifier() const { return m_identifier; }
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |     SharedWorkerScriptLoader
/home/bot/workspace/sources/wpewebkit/Source/WebCore/workers/shared/SharedWorkerScriptLoader.h:60:11: error: 'SharedWorkerScriptLoaderIdentifier' does not name a type; did you mean 'SharedWorkerScriptLoader'?
   60 |     const SharedWorkerScriptLoaderIdentifier m_identifier;
      |           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |           SharedWorkerScriptLoader
/home/bot/workspace/sources/wpewebkit/Source/WebCore/workers/shared/SharedWorkerScriptLoader.cpp: In constructor 'WebCore::SharedWorkerScriptLoader::SharedWorkerScriptLoader(const WTF::URL&, WebCore::SharedWorker&, WebCore::TransferredMessagePort&&, WebCore::WorkerOptions&&)':
/home/bot/workspace/sources/wpewebkit/Source/WebCore/workers/shared/SharedWorkerScriptLoader.cpp:39:7: error: class 'WebCore::SharedWorkerScriptLoader' does not have any field named 'm_identifier'
   39 |     : m_identifier(SharedWorkerScriptLoaderIdentifier::generate())
      |       ^~~~~~~~~~~~

[Pablo Saavedra]

Created attachment 450879 [details]
patch

[EWS]

[e-mail] does not have reviewer permissions according to https://raw.githubusercontent.com/WebKit/WebKit/main/metadata/contributors.json.

Rejecting attachment 450879 [details] from commit queue.

[Pablo Saavedra]

Committed r289107 (246804@trunk): <https://commits.webkit.org/246804@trunk>

[Radar WebKit Bug Importer]

<rdar://problem/88477288>

[Pablo Saavedra]

Just for reference, landed as unreviewed patch for the sake of simplicity .