Closed Bug 1369345 Opened 2 years ago Closed 2 years ago

Remove messageWakeupService

Categories

(Core :: DOM: Content Processes, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Iteration:
55.7 - Jun 12
Tracking Status
firefox55 --- fixed

People

(Reporter: florian, Assigned: florian)

References

(Blocks 1 open bug)

Details

(Whiteboard: [photon-performance])

Attachments

(1 file)

This was implemented in bug 591052 for Fennec. Bug 612718 to not build it in Firefox was wontfixed and it was packaged for Firefox in bug 726864 because "This has potential use in Firefox", but the mentioned used case (bug 674720) has been removed.

This shows up during Firefox startup and is a waste. We should not package this on Firefox. And given that I can't find any actual Fennec use case, I think we can just remove it completely.
Assignee: nobody → florian
Status: NEW → ASSIGNED
Alon, do you know if this is used anywhere for Fennec?
Flags: needinfo?(azakai)
Sorry, no idea - it's been many years since I worked on this stuff.
Flags: needinfo?(azakai)
Comment on attachment 8873373 [details] [diff] [review]
Remove messageWakeupService, r=

Review of attachment 8873373 [details] [diff] [review]:
-----------------------------------------------------------------

Nuke it from orbit!
Attachment #8873373 - Flags: review+
Pushed by florian@queze.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/55366075a57e
Remove messageWakeupService, r=nalexander.
https://hg.mozilla.org/mozilla-central/rev/55366075a57e
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Whiteboard: [photon-performance]
Iteration: --- → 55.7 - Jun 12
Flags: qe-verify?
Priority: -- → P1
Flags: qe-verify? → qe-verify-
You need to log in before you can comment on or make changes to this bug.