Closed
Bug 1300884
Opened 8 years ago
Closed 8 years ago
Remove the alarms API
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla51
Tracking | Status | |
---|---|---|
firefox51 | --- | fixed |
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
(Keywords: dev-doc-complete, site-compat)
Attachments
(1 file)
(deleted),
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8788613 -
Flags: review?(fabrice)
Updated•8 years ago
|
Attachment #8788613 -
Flags: review?(fabrice) → review+
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b85b4f18f956
Remove the alarms API; r=fabrice
Pushed by eakhgari@mozilla.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 eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7452e706f985
follow-up: Remove browser_alarms.js, landed on a CLOSED TREE
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/145930864704
follow-up: Remove the last import of AlarmService.jsm
Comment 6•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b85b4f18f956
https://hg.mozilla.org/mozilla-central/rev/b0c4d7caec6f
https://hg.mozilla.org/mozilla-central/rev/7452e706f985
https://hg.mozilla.org/mozilla-central/rev/77940cbf0c2a
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Comment 7•8 years ago
|
||
bugherder |
Updated•8 years ago
|
Keywords: dev-doc-needed
Comment 8•8 years ago
|
||
Posted the site compatibility doc: https://www.fxsitecompat.com/en-CA/docs/2016/alarm-api-has-been-removed/
Keywords: site-compat
Comment 9•8 years ago
|
||
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
Keywords: dev-doc-needed → dev-doc-complete
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•