Closed Bug 1300884 Opened 3 years ago Closed 3 years ago
Remove the alarms API
Instead of worrying about what to do with <http://searchfox.org/mozilla-central/source/dom/alarm/AlarmsManager.js#77>, we can just remove the entire thing.
Attachment #8788613 - Flags: review?(fabrice) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/b85b4f18f956 Remove the alarms API; r=fabrice
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b0c4d7caec6f follow-up: Remove files from package manifests too, landed on a CLOSED TREE
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/7452e706f985 follow-up: Remove browser_alarms.js, landed on a CLOSED TREE
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/145930864704 follow-up: Remove the last import of AlarmService.jsm
Posted the site compatibility doc: https://www.fxsitecompat.com/en-CA/docs/2016/alarm-api-has-been-removed/
This looks to have mostly been done already; the following had already been moved to the archive: https://developer.mozilla.org/en-US/docs/Archive/Firefox_OS/API/Alarm_API https://developer.mozilla.org/en-US/docs/Archive/Firefox_OS/API/Navigator/mozAlarms https://developer.mozilla.org/en-US/docs/Archive/Firefox_OS/API/MozAlarmsManager (and subpages) I also added obsolete notices to the relevant places on https://developer.mozilla.org/en-US/Apps/Fundamentals/User_notifications#Examples https://developer.mozilla.org/en-US/Apps/Fundamentals/User_notifications/Using_Alarms_to_notify_users But I've not removed these yet, as there is a separate bit of work around updating the apps zone. I also added a note to the Fx51 release notes: https://developer.mozilla.org/en-US/Firefox/Releases/51
You need to log in before you can comment on or make changes to this bug.