Bug 245727

Summary:	[ New Test ] (254620@main): [ Monterey wk2 ] imported/w3c/web-platform-tests/notifications/idlharness.https.any.sharedworker.html is a consistent failure
Product:	WebKit	Reporter:	Karl Rackler <rackler>
Component:	New Bugs	Assignee:	Karl Rackler <rackler>
Status:	RESOLVED CONFIGURATION CHANGED
Severity:	Normal	CC:	cdumez, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Description Karl Rackler 2022-09-27 06:50:09 PDT

Description:
imported/w3c/web-platform-tests/notifications/idlharness.https.any.sharedworker.html

The test was added at 254620@main and has been a consistent failure.

Reproduction steps:
I can reproduce this on 254902@main ToT.

Command: 
run-webkit-tests --iterations 50 --exit-after-n-failures 1 --exit-after-n-crashes-or-timeouts 1 -f --no-retry --release  imported/w3c/web-platform-tests/notifications/idlharness.https.any.sharedworker.html

History:
https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fnotifications%2Fidlharness.https.any.sharedworker.html&version_name=Monterey&limit=50000

DIFF:
@@ -61,6 +61,6 @@
 FAIL Notification interface: notification must inherit property "actions" with the proper type assert_equals: Unexpected exception when evaluating object expected null but got object "ReferenceError: Can't find variable: notification"
 FAIL Notification interface: notification must inherit property "close()" with the proper type assert_equals: Unexpected exception when evaluating object expected null but got object "ReferenceError: Can't find variable: notification"
 PASS NotificationEvent interface: existence and properties of interface object
-PASS ServiceWorkerRegistration interface: operation showNotification(DOMString, optional NotificationOptions)
-PASS ServiceWorkerRegistration interface: operation getNotifications(optional GetNotificationOptions)
+FAIL ServiceWorkerRegistration interface: operation showNotification(DOMString, optional NotificationOptions) assert_own_property: interface prototype object missing non-static operation expected property "showNotification" missing
+FAIL ServiceWorkerRegistration interface: operation getNotifications(optional GetNotificationOptions) assert_own_property: interface prototype object missing non-static operation expected property "getNotifications" missing

Comment 1 Radar WebKit Bug Importer 2022-09-27 06:50:58 PDT

<rdar://problem/100457001>

Comment 2 Karl Rackler 2022-09-27 06:52:32 PDT

I have marked this test as a consistent failure while this issue is investigated.

Comment 3 EWS 2022-09-27 07:03:54 PDT

Test gardening commit 254909@main (341d248459f6): <https://commits.webkit.org/254909@main>

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

Comment 4 Chris Dumez 2022-09-27 08:12:09 PDT

This feature is only available on macOS 13+ so this just needs a platform-specific baseline:
```
#if ENABLE(SERVICE_WORKER) && ENABLE(NOTIFICATIONS) \
    && ((PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 130000) || (PLATFORM(GTK) || PLATFORM(WPE)))
#if !defined(ENABLE_NOTIFICATION_EVENT)
#define ENABLE_NOTIFICATION_EVENT 1
#endif
#endif
```

Comment 5 Karl Rackler 2022-09-27 11:21:58 PDT

adding Monterey-specific baseline

Comment 6 Karl Rackler 2022-09-27 12:02:03 PDT

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

Comment 7 EWS 2022-09-27 12:05:13 PDT

Test gardening commit 254927@main (356c4a2e2a29): <https://commits.webkit.org/254927@main>

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

Comment 8 Karl Rackler 2022-09-27 15:09:27 PDT

Verified after adding Monterey-specific baseline at 254927@main test is consistently passing.