Closed Bug 1515945 Opened 6 years ago Closed 5 years ago

about:serviceworkers Fission support

Categories

(Core :: DOM: Service Workers, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla76
Fission Milestone Future
Tracking Status
firefox66 --- wontfix
firefox76 --- fixed

People

(Reporter: overholt, Assigned: asuth)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

Attachments

(1 file)

This might need some work due to the content process changes but we should also ensure it lives in the right process.
I think about:serviceworkers wants to go away in favor of about:debugging per bug 1273821. That said, you do make an excellent point that since about:serviceworkers gets loaded in a content process right now, it will not work with parent_intercept on. Perhaps this is a reason to accelerate the demise of about:serviceworkers.

Tentatively moving all bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to the "?" triage milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Fission Milestone: --- → ?

I feel like about:serviceworkers is already broken in Nightly, is that expected?

It is expected to be broken, yes. It asks the ServiceWorkerManager in whatever process it is in for the list of ServiceWorkers. With parent-intercept (dom.serviceWorkers.parent_intercept) enabled, only the parent process ServiceWorkerManager knows anything. Bug 1456995 enabled it on nightly. So in the content processes, they think there are no ServiceWorkers.

The options are:

This bug is not a Fission MVP blocker.

Fission Milestone: ? → Future

Andrew, do you want to fix this about:serviceworkers bug for Fission now or just fix bug 1273821 (remove about:serviceworkers and redirect to about:debugging#workers)?

Flags: needinfo?(bugmail)

Given that about:debugging is not available on Android, I would prefer if we kept (and fixed) about:serviceworkers.

(In reply to James Willcox (:snorp) (jwillcox@mozilla.com) (he/him) from comment #7)

Given that about:debugging is not available on Android, I would prefer if we kept (and fixed) about:serviceworkers.

Works for me!

Flags: needinfo?(bugmail)
Assignee: nobody → bugmail
Status: NEW → ASSIGNED

The parent is where the info is in the parent process. Without this fix,
about:serviceworkers misleadingly suggests there are no Service Workers
present at all. This is desired for mobile.

Pushed by bugmail@asutherland.org: https://hg.mozilla.org/integration/autoland/rev/bf05a0b449d8 Run about:serviceworkers in the parent process. r=perry
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: