-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Default silent to platform convention #194
Conversation
Thanks for the proposal Marcos, agreed that the default should be platform specific, and is an overall improvement. The change looks good to me, and happy to apply the changes from the chromium side of things. |
Small set of tests are coming as part of WebKit/WebKit#13912 (reviewed upstream) I’ll send them to WPT hopefully tomorrow. |
Implementer bugs filed. |
https://bugs.webkit.org/show_bug.cgi?id=256828 rdar://109390045 Reviewed by Brady Eidson. Makes the silent member on the NotificationOption dictionary default to null. This is then reflected by the silent attribute of the Notification interface. Spec change: whatwg/notifications#194 * LayoutTests/imported/w3c/web-platform-tests/interfaces/notifications.idl: * LayoutTests/imported/w3c/web-platform-tests/notifications/constructor-basic.https-expected.txt: * LayoutTests/imported/w3c/web-platform-tests/notifications/constructor-basic.https.html: * Source/WebCore/Modules/notifications/Notification.h: * Source/WebCore/Modules/notifications/Notification.idl: * Source/WebCore/Modules/notifications/NotificationOptions.idl: Canonical link: https://commits.webkit.org/264397@main
Tests sent at web-platform-tests/wpt#40160 (reviewed upstream) |
… to null, a=testonly Automatic update from web-platform-tests Notifications: test that silent defaults to null For whatwg/notifications#194. -- wpt-commits: ac2c4415f6ea3f0d08a2d19f0e104ce447054db1 wpt-pr: 40160
… to null, a=testonly Automatic update from web-platform-tests Notifications: test that silent defaults to null For whatwg/notifications#194. -- wpt-commits: ac2c4415f6ea3f0d08a2d19f0e104ce447054db1 wpt-pr: 40160
… to null, a=testonly Automatic update from web-platform-tests Notifications: test that silent defaults to null For whatwg/notifications#194. -- wpt-commits: ac2c4415f6ea3f0d08a2d19f0e104ce447054db1 wpt-pr: 40160 UltraBlame original commit: e3ed9dae6c2e719eb3ca042485d5493caac92613
… to null, a=testonly Automatic update from web-platform-tests Notifications: test that silent defaults to null For whatwg/notifications#194. -- wpt-commits: ac2c4415f6ea3f0d08a2d19f0e104ce447054db1 wpt-pr: 40160 UltraBlame original commit: e3ed9dae6c2e719eb3ca042485d5493caac92613
… to null, a=testonly Automatic update from web-platform-tests Notifications: test that silent defaults to null For whatwg/notifications#194. -- wpt-commits: ac2c4415f6ea3f0d08a2d19f0e104ce447054db1 wpt-pr: 40160 UltraBlame original commit: e3ed9dae6c2e719eb3ca042485d5493caac92613
Closes #193
(See WHATWG Working Mode: Changes for more details.)
Preview | Diff