end icon indicating copy to clipboard operation
end copied to clipboard

Firefox spuriously fallbacks to it's own notifications

Open VixieTSQ opened this issue 1 year ago • 6 comments

I have it so when I click on my notification in eww, it'll call end close <id>. But when the timeout for the notification is up and end tries to close it, it fails and crashes because the notification is already closed

VixieTSQ avatar Feb 03 '24 23:02 VixieTSQ

Or sorry, not a crash. But it makes firefox think it crashed because it sends an error back to it over DBus (my educated guess). So firefox starts using it's own notification popups again.

VixieTSQ avatar Feb 03 '24 23:02 VixieTSQ

Interesting. The closing after the timeout is implemented as a filter over the list of notifications, so if a notification was already closed/removed before, it should be a no-op. But we currently don't send the NotificationClosed signal; maybe that causes issues in some cases. On the other hand, when notifications simply time out, it seems to work even without the signal.

It's late here, I will try to reproduce this tomorrow.

lucalabs-de avatar Feb 04 '24 00:02 lucalabs-de

I cannot reproduce this by closing notifications manually. However, it seems that sometimes Firefox notices when I restart end to apply changes and then falls back to its own notifications. Restarting Firefox while end is running fixes this. Is this maybe what happened in your case as well?

lucalabs-de avatar Feb 04 '24 13:02 lucalabs-de

I cannot reproduce this by closing notifications manually. However, it seems that sometimes Firefox notices when I restart end to apply changes and then falls back to its own notifications. Restarting Firefox while end is running fixes this. Is this maybe what happened in your case as well?

Very odd. Now I can't reproduce it either. I will keep this open for bit while I observe to see when firefox breaks.

VixieTSQ avatar Feb 04 '24 19:02 VixieTSQ

I cannot reproduce this by closing notifications manually. However, it seems that sometimes Firefox notices when I restart end to apply changes and then falls back to its own notifications. Restarting Firefox while end is running fixes this. Is this maybe what happened in your case as well?

It's still definitely happening, I just can't say why. It does happen even when I'm not messing around with end. And end doesn't even print anything when firefox stops liking it. I'm stumped

VixieTSQ avatar Feb 05 '24 03:02 VixieTSQ

Maybe this is related to all the missing signals we're not sending.

lucalabs-de avatar Feb 07 '24 09:02 lucalabs-de