Closed
Bug 1369345
Opened 8 years ago
Closed 8 years ago
Remove messageWakeupService
Categories
(Core :: DOM: Content Processes, enhancement, P1)
Core
DOM: Content Processes
Tracking
()
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: florian, Assigned: florian)
References
(Blocks 1 open bug)
Details
(Whiteboard: [photon-performance])
Attachments
(1 file)
6.77 KB,
patch
|
nalexander
:
review+
|
Details | Diff | Splinter Review |
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 | ||
Comment 1•8 years ago
|
||
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → florian
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•8 years ago
|
||
Alon, do you know if this is used anywhere for Fennec?
Flags: needinfo?(azakai)
Comment 3•8 years ago
|
||
Sorry, no idea - it's been many years since I worked on this stuff.
Flags: needinfo?(azakai)
Comment 4•8 years ago
|
||
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.
Assignee | ||
Updated•8 years ago
|
Blocks: photon-startup
Comment 6•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee | ||
Updated•8 years ago
|
Whiteboard: [photon-performance]
Updated•8 years ago
|
Iteration: --- → 55.7 - Jun 12
Flags: qe-verify?
Priority: -- → P1
Assignee | ||
Updated•8 years ago
|
Flags: qe-verify? → qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•