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




6 years ago
22 days ago


(Reporter: baku, Unassigned)



Firefox Tracking Flags

(Not tracked)



(1 attachment)

Comment hidden (empty)

Comment 1

6 years ago
Created attachment 690838 [details] [diff] [review]
Attachment #690838 - Flags: review?(rlin)


6 years ago
OS: Linux → All
Hardware: x86_64 → All

Comment 2

6 years ago
I forgot to change the UUID of the IDL but I'll wait for comments before submitting a new patch.
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.
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

6 years ago
This patch it's just about the usage of the content audio channel if multiple apps use the radio concurrently.
Comment on attachment 690838 [details] [diff] [review]

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.
Attachment #690838 - Flags: review?(rlin) → review?(jones.chris.g)
Andrea, do you think this code has value while we await bug 820241?  What use cases do you have in mind?
Flags: needinfo?(amarchesini)

Comment 8

6 years ago
> 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.
Flags: needinfo?(amarchesini)
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 :).

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.
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.