1201090 - Clicking on Web Notifications spawned by serviceWorker doesn't open specified tab

Status: RESOLVED DUPLICATE of bug 1130687

(Firefox :: Site Permissions, defect)

Description

[Bill Maggs (bmaggs)]

Reproduce: 
Currently in FF 42 and 43, with Push Messages turned on, the push message only brings the focus to the page that sent the message if the tab is open. If the user closes the tab, they are stuck with a message that won't take them anywhere. This is confusing for the user. In Chrome, clicking on the message after the sending page is closed reopens the page. 


To reproduce: 

0. set System preference on Notifications for FF to Alerts (won't disappear after a few seconds) 

1. send push notification message to FF (can use johnme-gcm.appspot.com as a sample app)

2. Close tab with johnme app

3. If another tab is open, clicking on message brings up last foregrounded tab. On OSx if there is no tab open, the message is not clickable. On Windows if there is no tab open, clicking the message brings you to about: home or the new tab page. 


Expected behavior: receiving a Push Message allows the user to click on the message and get back to the page that the user has given permission to send messages. For example, doing the above steps in Chrome reopens the tab that the message connects to.

Comment 1

[Lina Butler [:lina]]

The service worker uses `clients.openWindow()` to open the tab (https://johnme-gcm.appspot.com/chat/sw.js), which I don't think we support yet? https://dxr.mozilla.org/mozilla-central/source/dom/workers/ServiceWorkerClients.cpp?offset=0#278-289

Comment 2

[Edwin Wong [:edwong]]

:kit looks like openWindow() has been worked on, looks like there's more to do: https://bugzilla.mozilla.org/show_bug.cgi?id=1130687

Comment 3

[Andrew Overholt [:overholt]]

Bill and Rory: here's something that isn't currently on our SW-v1 radar.

Comment 4

[Doug Turner (:dougt)]

overholt, can you find an owner for this?

Comment 5

[Andrew Overholt [:overholt]]

Sorry, forgot to update here.  I emailed some people but we're trying to land Service Worker network interception in 43 and this would take away from that.  Is this critical to land in 43/42 (Aurora/Beta)?

Comment 6

[Doug Turner (:dougt)]

Yeah, Andrew.  Without this, displaying an UI from a push notification doesn't work. :?

Comment 7

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

This needs work from someone who knows the window/PContent/PBrowser/e10s stuff, especially if we are going to ship this with e10s enabled. For non-e10s it *may* be easier. I tried taking a stab at this (patches in Bug 1130687) but no dice. The fact that any of our window opener code relies on there being an existing window around (based on the premise that such calls come from window.open() or similar) complicates everything when service workers are involved (where there may not be a window present). Based on this, I don't feel confident in assigning this to anyone not familiar with that code.