Closed
Bug 1300884
Opened 7 years ago
Closed 7 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)
|
92.01 KB,
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•7 years ago
|
||
Attachment #8788613 -
Flags: review?(fabrice)
|
||
Updated•7 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•7 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: 7 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
|
||
Comment 7•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/145930864704
|
||
Updated•7 years ago
|
Keywords: dev-doc-needed
|
||
Comment 8•7 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•7 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•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•