Closed Bug 1003063 Opened 10 years ago Closed 10 years ago

Remove mozContentEvent for notifications

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::System
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: gerard-majax, Assigned: gerard-majax)

References

Details

(Whiteboard: [systemsfe] [p=1])

Attachments

(1 file)

Link to Github https://github.com/mozilla-b2g/gaia/pull/18856
46 bytes, text/x-github-pull-request
alive
: review+
mikehenrty
: review+
Details | Review 
After landing bug 963234 we can remove mozContentEvent usage in Gaia system app for notifications, and completely switch to mozNotificationEvent.
Please find attached a link to the github pull request that removes those old events. Travis failure is expected until gecko part for bug 963234 lands.
Attachment #8415817 - Flags: review?(mhenretty)
Attachment #8415817 - Flags: review?(alive)
No longer blocks: 963234
Depends on: 963234
Whiteboard: [systemsfe] [p=1]
Comment on attachment 8415817 [details] [review]
Link to Github https://github.com/mozilla-b2g/gaia/pull/18856

Retriggered the tests after bug 963234 landed on m-c. Nice and green.
Attachment #8415817 - Flags: review?(mhenretty) → review+
Attachment #8415817 - Flags: review?(alive) → review+
All green again, time to land.
https://github.com/mozilla-b2g/gaia/commit/77e9b2f5c98b5e9ed2712bb0526a4d7bbcc4a44d
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: