820313 - The visibility of the FMRadio component can be depend by more than 1 app

Status: NEW

(Core :: DOM: Core & HTML, defect, P5)

Comment 1

[Andrea Marchesini [:baku]]

Attachment: patch

Comment 2

[Andrea Marchesini [:baku]]

I forgot to change the UUID of the IDL but I'll wait for comments before submitting a new patch.

Comment 3

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Hm, we had planned to review this for bug 820241.  There are many bad things that multiple clients can do right now.  But this is fine.

Comment 4

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Although on second thought I'm not sure this patch is going to improve the experience of multiple apps trying to use the radio concurrently.

Comment 5

[Andrea Marchesini [:baku]]

This patch it's just about the usage of the content audio channel if multiple apps use the radio concurrently.

Comment 6

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Comment on attachment 690838 [details] [diff] [review]
patch

I don't have the right to set r+, transfer to proper one.
Maybe only allow one client to use fmradio is better for avoiding such kind of competition.

Comment 7

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Andrea, do you think this code has value while we await bug 820241?  What use cases do you have in mind?

Comment 8

[Andrea Marchesini [:baku]]

> Maybe only allow one client to use fmradio is better for avoiding such kind
> of competition.

If we do this, this patch is not needed.
In general, my concern is that multiple apps can use FMRadio but the visibility is a boolean flag set by one of them.

Comment 9

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

That's true, but the enabled-ness and channel are also set by one of them.

If you don't have some use cases in mind, I'd prefer we take this patch as a starter for bug 820241 :).

Comment 10

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.