Closed Bug 791869 Opened 7 years ago Closed 7 years ago

Remove services/notifications

Categories

(Cloud Services :: WebPush, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla20

People

(Reporter: gps, Assigned: gps)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #754062 +++

I'm pretty sure that services/notifications isn't going anywhere and can safely be removed from the tree. If we ever need it, it's there in the commit history. It's just a skeleton for the XPCOM foo anyway.
Rubber stamp, please.
Assignee: nobody → gps
Status: NEW → ASSIGNED
Attachment #698451 - Flags: review?(rnewman)
Attachment #698451 - Flags: review?(rnewman) → review+
https://hg.mozilla.org/mozilla-central/rev/7647b05aa3f0
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in services]
Target Milestone: --- → mozilla20
Should this one http://mxr.mozilla.org/comm-central/source/mozilla/browser/installer/package-manifest.in#590
590 #ifdef MOZ_SERVICES_NOTIFICATIONS
591 @BINPATH@/defaults/pref/services-notifications.js
592 #endif

also get removed?
Yup. There is also a reference in configure.in that should be killed.
Blocks: 827683
You need to log in before you can comment on or make changes to this bug.