Closed
Bug 1169229
Opened 9 years ago
Closed 9 years ago
[Statusbar] Fix dependency between the statusbar and the icons
Categories
(Firefox OS Graveyard :: Gaia::System::Status bar, Utility tray, Notification, defect)
Firefox OS Graveyard
Gaia::System::Status bar, Utility tray, Notification
ARM
Gonk (Firefox OS)
Tracking
(Not tracked)
RESOLVED
FIXED
2.2 S14 (12june)
People
(Reporter: apastor, Assigned: apastor)
References
Details
(Whiteboard: [systemsfe])
Attachments
(1 file)
Right now we are listening to iconcreated/iconrendered events in the statusbar for handling the priority of them. We should make sure we receive those events, even when the icon is instantiated before the statusbar.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → apastor
Comment 1•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8613955 -
Flags: review?(alive)
Comment 2•9 years ago
|
||
Comment on attachment 8613955 [details] [review] [gaia] albertopq:1169229-statusbar-dependency > mozilla-b2g:master Awesome, I didn't expect this kind of usage of Service... but it looks good. I will say it would be nice to test these as well: 1. Statusbar registers to Service for iconContain before BaseIcon calls render 2. Statusbar registers to Service for iconContain after BaseIcon calls render 3. BaseIcon created before Statusbar registers to Service for iconCreated 4. BaseIcon created after Statusbar registers to Service for iconCreated But this test may be difficult to write, so it's up 2 u.
Attachment #8613955 -
Flags: review?(alive) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Done! master: https://github.com/mozilla-b2g/gaia/commit/38f2e6bfa819b1417c3cbd4299a8121f5fe4f2a0
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Target Milestone: --- → 2.2 S14 (12june)
You need to log in
before you can comment on or make changes to this bug.
Description
•