Closed Bug 1300884 Opened 8 years ago Closed 8 years ago

Remove the alarms API

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

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)

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.
Attached patch Remove the alarms API (deleted) — Splinter Review
Attachment #8788613 - Flags: review?(fabrice)
Attachment #8788613 - Flags: review?(fabrice) → review+
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
Blocks: 1350169
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: