948349 - Convert System to new Notification API

Status: RESOLVED DUPLICATE of bug 1095109

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

Description

[:gerard-majax]

apps/system/js/battery_manager.js:
  - sending notification of power saving mode
  
apps/system/js/bluetooth_transfer.js:
  - sending notification of bluetooth transfers
    - started
    - confirmation
    - sent finished successfully
    - receive finished successfully
    - sent finished with error
    - receive finished with error
    - multiple transfers report
  
apps/system/js/icc_worker.js:
  - sending notification from STK CMD_SET_UP_IDLE_MODE_TEXT

Comment 1

[:gerard-majax]

There is also a direct use in: apps/system/js/screenshot.js which should be quite easy to convert, it's just for notifying of screenshots.

Comment 2

[:gerard-majax]

(In reply to Alexandre LISSY :gerard-majax from comment #1)
> There is also a direct use in: apps/system/js/screenshot.js which should be
> quite easy to convert, it's just for notifying of screenshots.

So it seems someone already fixed this ...

Comment 3

[:gerard-majax]

(In reply to Alexandre LISSY :gerard-majax from comment #0)
> apps/system/js/icc_worker.js:
>   - sending notification from STK CMD_SET_UP_IDLE_MODE_TEXT

This one also seems to have been fixed.

Comment 4

[:gerard-majax]

(In reply to Alexandre LISSY :gerard-majax from comment #2)
> (In reply to Alexandre LISSY :gerard-majax from comment #1)
> > There is also a direct use in: apps/system/js/screenshot.js which should be
> > quite easy to convert, it's just for notifying of screenshots.
> 
> So it seems someone already fixed this ...

Looks like it was done in bug 971546.

Comment 5

[:gerard-majax]

(In reply to Alexandre LISSY :gerard-majax from comment #3)
> (In reply to Alexandre LISSY :gerard-majax from comment #0)
> > apps/system/js/icc_worker.js:
> >   - sending notification from STK CMD_SET_UP_IDLE_MODE_TEXT
> 
> This one also seems to have been fixed.

Fixed by bug 873397.

Comment 6

[Zibi Braniecki [:zbraniecki][:gandalf]]

Hey guys, 

:gerard-majax made me aware that this bug exists. We found a need for a Notification wrapper for L10n purposes and we want to reuse NotificationHelper. See bug 1095109. Because it also moves NotificationHelper to use W3C Notification API, I'm marking this bug as a dupe of that one.