Closed Bug 1016918 Opened 10 years ago Closed 7 years ago

The Topic Names for NotifyObservers in AudioChannelService are Not Accurate for Meaning

Categories

(Firefox OS Graveyard :: AudioChannel, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: mchen, Unassigned)

Details

Currently there are three topic names created to NotifyObservers and listed as below; 1. audio-channel-process-changed Used to notify ProcessPriorityManager to reset priority of a specific process. 2. audio-channel-changed Used to notify shell.js that the highest audio channel type (including audio channels in foreground or background) is changed. 3. visible-audio-channel-changed Used to notify shell.js that the highest audio channel type in the foreground is changed. The naming of these topics is not good to understand and we may need to refind these topics to more generic purpose. ex: audio-channel-process-changed -> audiochannel-added-removed audio-channel-changed -> audiochannel-highest-type-changed audio-channel-changed -> visible-audiochannel-highest-type-changed
Audio channel code has been removed, this is not relevant anymore.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.