[Stingray][Notification] Establish IAC between smart-system and notification app for notification messages.

RESOLVED FIXED

Status

Firefox OS
Gaia
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: rexboy, Assigned: rexboy)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [ft:conndevices])

Attachments

(1 attachment)

This bug aims for IAC connection for notification messages.
In specific, smart-system must send the following events to notification app in appropriate forms:
1.desktop-notification
2.desktop-notification-close
3.resend count

and receive following events from notification app in appropriate forms, then feeding them to Gecko.
1.desktop-notification-click
2.desktop-notification-show
3.desktop-notification-close
Assignee: nobody → rexboy
Created attachment 8630862 [details] [review]
patch

John: may you help review the system app part.
Yifan: may you confirm that these IAC matches your need.

Thank you guys!
And I'll add tests (here or another bug) once things get confirmed.
Attachment #8630862 - Flags: review?(yliao)
Attachment #8630862 - Flags: review?(im)
Comment on attachment 8630862 [details] [review]
patch

Looks good to me. I found some errors. Please update them before commit.
Attachment #8630862 - Flags: review?(im) → review+
Comment on attachment 8630862 [details] [review]
patch

LGTM! Thanks! Do we need a way for a notification widget to ask for resending notifications?
Attachment #8630862 - Flags: review?(yliao) → review+
Thank you guys!
I'm not sure if we need to ask for resending. It would be raised once booting.
If we need that for some managing requirements, we can do it on another bug.
master:

https://github.com/mozilla-b2g/gaia/commit/8bca5aa18818bd1ea6de13991d5954371eaec1b8
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.