Closed
Bug 1369345
Opened 7 years ago
Closed 7 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•7 years ago
|
||
|
|
Assignee | |
Updated•7 years ago
|
Assignee: nobody → florian
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•7 years ago
|
||
Alon, do you know if this is used anywhere for Fennec?
Flags: needinfo?(azakai)
|
||
Comment 3•7 years ago
|
||
Sorry, no idea - it's been many years since I worked on this stuff.
Flags: needinfo?(azakai)
|
||
Comment 4•7 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•7 years ago
|
Blocks: photon-startup
|
||
Comment 6•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/55366075a57e
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
|
|
Assignee | |
Updated•7 years ago
|
Whiteboard: [photon-performance]
|
||
Updated•7 years ago
|
Iteration: --- → 55.7 - Jun 12
Flags: qe-verify?
Priority: -- → P1
|
|
Assignee | |
Updated•7 years ago
|
Flags: qe-verify? → qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•