1669224 - clicking on Yahoo!'s "new email" overlay notification does not open the corresponding email-tab

Status: UNCONFIRMED

(Core :: DOM: Notifications, defect, P3)

Description

[Dan]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:81.0) Gecko/20100101 Firefox/81.0

Steps to reproduce:

Open several tabs, one of them being your Yahoo! Email account (inbox).
Now switch to a different tab.
next receive an email to that Yahoo! Mail inbox.

Actual results:

Overlay-notification in bottom right corner appears, displaying new email-title and some content. When clicking on the notification, it just vanishes, nothing more happens.

Expected results:

it should also automatically open the tab of the Yahoo! Email-inbox (and maybe even display the corresponding email even, if possible.)

Comment 1

[Jens Stutte [:jstutte]]

Simon, can you triage this?

Comment 2

[Simon Giesecke [:sg] [he/him]]

I don't think this belongs into Push Notifications (since the push notification is reported to occur), I reassigned it to Notification and Alerts.

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

It seems like the point this would be going wrong is in Client.focus or Clients.openWindow. There's a chance this could be a bug in the Yahoo ServiceWorker losing the causal user interaction chain, or a cross-browser inconsistency for a time-based grant, etc. I think it makes sense to track this in the ServiceWorkers component for now, although it's possible the bug would be in the user interaction tracking of push notifications.

I'm going to call this something that the about:serviceworkers logging in bug 1506892 should be able to help us diagnose in the near/medium future. Marking this as depending on that, and I've refreshed my yahoo mail credentials so I can try and reproduce it.

Comment 4

[Andrew Sutherland [:asuth] (he/him)]

I'll try and reproduce on my account since we've had some recent fixes currently on nightly (imminently on beta) that may have fixed this if this hadn't otherwise been fixed before then.

I've removed the dep because we're moving to using profiler instead of adding distinct tooling in about:serviceworkers.

Comment 5

[Kagami Rosylight [:saschanaz] (they/them)]

(Adding on comment #4: we had bug 1874511 (service worker was going to bed without properly unregistering notification observer), that issue was fixed in bug 1928702.)