Closed
Bug 1159610
Opened 9 years ago
Closed 9 years ago
[Callscreen] Use the telephony's audio channel api to make telephony channel manageable by System app
Categories
(Firefox OS Graveyard :: Gaia::Dialer, defect)
Tracking
(b2g-master fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-master | --- | fixed |
People
(Reporter: dkuo, Assigned: dkuo)
References
Details
Attachments
(1 file)
To enable the new audio channel service in gaia, we need to let System app know which app is using the telephony channel, because the telephony channel does not use media element to produce sounds, there will be no event when the telephony channel activates.
Bug 1129882 will introduce a new api for telephony's audio channel, it's:
|Telephony.ownAudioChannel()|
So after/before the api landed, we need to call it in Callscreen app so that the System app is able to manage the telephony audio channel.
Comment 1•9 years ago
|
||
(In reply to Dominic Kuo [:dkuo] from comment #0)
> So after/before the api landed, we need to call it in Callscreen app so that
> the System app is able to manage the telephony audio channel.
Can you elaborate when this call has to be made? At the beginning of a call?
Assignee | ||
Comment 2•9 years ago
|
||
(In reply to Gabriele Svelto [:gsvelto] from comment #1)
> Can you elaborate when this call has to be made? At the beginning of a call?
The main goal to call this api is to let the System's audio channel service knows about Callscreen will be using the telephony channel, we can say it's the initialization of audio channel. So if possible, it's better to do it before a call actually comes, not to wait the ril state changes. But I don't know the Callscreen code well, you guys should be able to make better judgement than me, if there are concerns, like performance regression.
Comment 3•9 years ago
|
||
I think this should be implemented before landing the new architecture of the audio channel.
Since the JS can detect whether the specific function exists, if it doesn't, we can ignore it.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → dkuo
Assignee | ||
Comment 4•9 years ago
|
||
Gabriele, would you please review this? thanks.
Attachment #8616605 -
Flags: review?(gsvelto)
Comment 5•9 years ago
|
||
Comment on attachment 8616605 [details]
patch
As per comment 3 there's no problem in landing this as is.
Attachment #8616605 -
Flags: review?(gsvelto) → review+
Assignee | ||
Comment 6•9 years ago
|
||
Thanks Gabriele :)
master: https://github.com/mozilla-b2g/gaia/commit/314cc9b2b56cfef14875101f2350aa3be15e7cae
You need to log in
before you can comment on or make changes to this bug.
Description
•