Closed
Bug 1879112
Opened 5 months ago
Closed 2 months ago
Notification::CloseInternal happens after permissionless `new Notification()`
Categories
(Core :: DOM: Notifications, defect, P3)
Core
DOM: Notifications
Tracking
()
RESOLVED
FIXED
127 Branch
Tracking | Status | |
---|---|---|
firefox127 | --- | fixed |
People
(Reporter: saschanaz, Assigned: saschanaz)
References
Details
Attachments
(1 file)
But it's not shown, so no point to close it.
The behavior happens when running ./mach wpt /notifications/constructor-basic.https.html /notifications/constructor-invalid.https.html
. (The notifications triggered from -basic without the permission are closed while -invalid is running. 🤔)
Assignee | ||
Comment 1•5 months ago
|
||
Updated•5 months ago
|
Severity: -- → S3
Priority: -- → P3
Updated•5 months ago
|
Attachment #9379062 -
Attachment description: WIP: Bug 1879112 → WIP: Bug 1879112 - Check permission earlier and mark as closed if failed
Assignee | ||
Comment 2•4 months ago
|
||
It's crashing on CI (specifically with ./mach wpt /notifications/getnotifications-across-processes.https.window.html /notifications/historical.any.worker.html
and the fix depends on bug 1879934.
Depends on: 1879934
Updated•4 months ago
|
Attachment #9379062 -
Attachment description: WIP: Bug 1879112 - Check permission earlier and mark as closed if failed → Bug 1879112 - Check permission earlier and mark as closed if failed r=asuth
Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2b3b6edc0cff Check permission earlier and mark as closed if failed r=asuth
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/44881 for changes under testing/web-platform/tests
Comment 5•4 months ago
|
||
Backed out for causing assertion failures @ widget/windows/ToastNotification.cpp
Backout link: https://hg.mozilla.org/integration/autoland/rev/73ca2889eeda977c4f28be1c7db646ddc6cd38a6
Flags: needinfo?(krosylight)
Upstream PR merged by moz-wptsync-bot
Updated•4 months ago
|
Attachment #9379062 -
Attachment description: Bug 1879112 - Check permission earlier and mark as closed if failed r=asuth → WIP: Bug 1879112 - Check permission earlier and mark as closed if failed r=asuth
Assignee | ||
Updated•4 months ago
|
Flags: needinfo?(krosylight)
Assignee | ||
Updated•2 months ago
|
Component: DOM: Push Subscriptions → DOM: Notifications
Updated•2 months ago
|
Attachment #9379062 -
Attachment description: WIP: Bug 1879112 - Check permission earlier and mark as closed if failed r=asuth → Bug 1879112 - Check permission earlier and mark as closed if failed r=asuth
Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bf3735b436f1 Check permission earlier and mark as closed if failed r=asuth
Comment 8•2 months ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 2 months ago
status-firefox127:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 127 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•