Bug 238520 – [Playstation] Fix build break after r292041 when ENABLE_SERVICE_WORKER is off

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Jigen Zhou]

There was 2 additional build errors occur after r292041 (the fix to r291979) ( if NOTIFICATIONS is also off:

C:\Working\neko_dir\neko-ps\Source\WebKit\WebProcess/Notifications/WebNotificationManager.cpp(153,93): error : member access into incomplete type 'WebCore::Notif
ication' [C:\Working\neko_dir\neko-ps\WebKitBuild\Debug-PS5\Source\WebKit\WebKit.vcxproj]
      LOG(Notifications, "WebProcess %i going to show notification %s", getpid(), notification.identifier().toString().utf8().data());
                                                                                              ^
  C:\Working\neko_dir\neko-ps\WebKitBuild\Debug-PS5\WebCore\PrivateHeaders\WebCore/NotificationClient.h(39,7): note: forward declaration of 'WebCore::Notificatio
  n'
  class Notification;
        ^
  1 error generated.

:\Working\neko_dir\neko-ps\Source\WebKit\UIProcess/WebsiteData/WebsiteDataStore.cpp(2114,18): error : no member named 'setServiceWorkerOpenWindowCompletionCallb
ack' in 'WebKit::WebPageProxy' [C:\Working\neko_dir\neko-ps\WebKitBuild\Debug-PS5\Source\WebKit\WebKit.vcxproj]
          newPage->setServiceWorkerOpenWindowCompletionCallback(WTFMove(innerCallback));
          ~~~~~~~  ^
  1 error generated.

[Jigen Zhou]

The error for NOTIFICATIONS has been fixed by r291994.

[Jigen Zhou]

Created attachment 456059 [details]
Patch

[Darin Adler]

Comment on attachment 456059 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=456059&action=review

> Source/WebKit/ChangeLog:3
> +        [Playstation] Fix build break after r292041 when ENABLE_SERVICE_WORKER is off

This bug name does not need a platform prefix, and should not have one.

[EWS]

Committed r292081 (249008@main): <https://commits.webkit.org/249008@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 456059 [details].

[Radar WebKit Bug Importer]

<rdar://problem/91013132>