Open
Bug 1369476
Opened 7 years ago
Updated 1 year ago
PushEvent is not fired in e10s mode when there are no content processes running
Categories
(Core :: DOM: Service Workers, defect, P3)
Core
DOM: Service Workers
Tracking
()
NEW
People
(Reporter: bkelly, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
We should start a process in this case: https://dxr.mozilla.org/mozilla-central/source/dom/push/PushNotifier.cpp#278 We can use this method: ContentParent::GetNewOrUsedBrowserProcess() Currently we basically just ignore push events on mac when firefox is open, but we have no content processes because all the windows have been closed. I feel like there is already a bug on this, but I can't find it.
Comment 1•7 years ago
|
||
We should fix this. When does it occur? Only having about:<something> open? On macOS when you have all windows closed but Firefox is still running?
Flags: needinfo?(bkelly)
Priority: -- → P2
Reporter | ||
Comment 2•7 years ago
|
||
Macos with no windows is the main case I'm aware of.
Flags: needinfo?(bkelly)
Comment 3•7 years ago
|
||
Having just private browsing windows and receiving push messages might be affected by this bug on other platforms. Anyhow, fixing this would require being able to keep content processes running for SW instances. To do that we need some sort of abstraction over windows and workers that are considered for the lifetime of a ContentParent. I thought Ben's client patches partially address this - can we fix this now without resorting to a hack?
Reporter | ||
Comment 4•7 years ago
|
||
Oh right. Its not just starting the process, but keeping it alive. This probably needs to wait for our multi-e10s refactor.
Depends on: ServiceWorkers-e10s
Comment 6•5 years ago
|
||
Moving to p3 because no activity for at least 1 year(s). See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Updated•1 year ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•