If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[Clock] Ringtone channel issue on background music

NEW
Unassigned

Status

Firefox OS
AudioChannel
P2
normal
5 years ago
3 years ago

People

(Reporter: leo.bugzilla.gecko, Unassigned)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(tracking-b2g:backlog)

Details

(Whiteboard: [AUDIO_COMPETING] [UX_TRIAGED])

(Reporter)

Description

5 years ago
[scenario]
playing background music - select clock ringtone preview - pause music - play clock ringtone  - slelect another ringtone - resume music shortly - play clock ringtone

[issue]
Don`t resume music shortly when select another ringtone.

[suggesion]
This case should need senario improvements. Because gecko/qualcomn can`t adjust timing only for this case, You should change scenario that don`t resume music when user select the other ringtone list. In a diffrent way, It takes interval time when select the ringtone list. 

Thanks.
Looks like a timing issue for switching channel when pause() a playing ringtone and set a new ringtone to play() in alarm channel.

Macro,
In the scenario, could we set a new ringtone to play() directly without pause() the playing ringtone? I'm not sure that it is a workaround or not. Do you have any idea?
Note: depend on Bug 829409

Updated

5 years ago
QA Contact: jshih → whsu
(Reporter)

Comment 3

5 years ago
May I check when we receve all patch include Bug 829409? 
This patch isn`t released in the recent version.
If you need more modification, let me know progress.
(Reporter)

Comment 4

5 years ago
dose anybody recomment about your patch?
I wanna patch date or version.
leo currently use ver 1.01.
(Reporter)

Updated

5 years ago
blocking-b2g: --- → leo?
More information would greatly help in making the decision whether or not to block the release on this.  Please elaborate on comment 0 and re-nominate if it's still a stop-ship issue.  Thanks.
blocking-b2g: leo? → -
Flags: needinfo?(leo.bugzilla.gecko)
(Reporter)

Comment 6

5 years ago
We like to avoid resuming the music shortly  when another ringtone is pressed. 
Because Music is automatically resume or pause by AudioChannelService in gecko layer, It is not easy to modify that music isn`t resume in gaia`s ringtone list. 
So i woluld recommand how to add the exception API for playing another ringtone directly.

Let you know this`s priority after discussing with leo manager.
Flags: needinfo?(leo.bugzilla.gecko)
(Reporter)

Comment 7

5 years ago
We think it is not a serious issue after discussing about it. We need the final decision of quality management department. We will keep status for a while.
thanks.
I'd like to get a UX opinion on this bug -- I find the current state acceptable and would support resolving as worksforme. 

Admittedly, there is some strangeness here where some of the sounds are very short (Classic Pulse) and others are quite a bit longer (Classic Pulse Progressive) so you don't have great predictability about when exactly the tone will end and the music will restart.
Flags: needinfo?(jhuang)

Comment 9

4 years ago
Dylan, I think this issue is about the audio competing policy, currently there are gaps between the alarm(clock) and content(music) channels while switching one to another. We can probably use some tricks in clock to avoid these gaps, but similar scenario will also happen like picking ringtones in settings(Setting > Sound > Ringer) when playing music in the background(Currently those two channels will be mixed together because ringtones app did not set the mozAudioChannelType properly, I will file a new bug later).

So let's keep this bug open and change the component to AudioChannel, next week we will have a workshop in Taipei office to discuss about, how to improve the audio competing policy, we should have some result and will update here, thanks.
Component: Gaia::Clock → AudioChannel

Updated

4 years ago
See Also: → bug 958470

Comment 10

4 years ago
In UX perspective, I suggest that when the user enters ringtone preview, the background music should always pause until the user leaves the page. The behaviour of selecting ringtones / ringer shouldn't be interrupted by background music.
Flags: needinfo?(jhuang)

Updated

4 years ago
Whiteboard: [AUDIO_COMPETING]

Comment 11

4 years ago
(In reply to Juwei Huang from comment #10)
> In UX perspective, I suggest that when the user enters ringtone preview, the
> background music should always pause until the user leaves the page. The
> behaviour of selecting ringtones / ringer shouldn't be interrupted by
> background music.

Thanks, Juwei, and after the offline meeting with ux team and gecko devs, we also agreed with the behaviour in comment 10.

Due to the current audio competing policy, we are unable to fix it without some workarounds, so I turned to study the web audio api and see if the technique can fix it. By doing some experiments, fortunately, I found using the web audio can fix it perfectly! and the same technique fixes bug 958470 as well, so I am going to write a patch, though this is not a blocker but fixing this can make big changes on the user experiences.
Assignee: nobody → dkuo

Comment 12

4 years ago
Update:

Since bug 959020 is requesting the feature that users can select user-defined ringtone to alarm, also the ringtones app is integrating the music pick activity, to be able to select ringtones from user's collection in bug 960329, in theory, clock should also pick the alarm tones from the same ringtones app, so this audio competing issue will be transferred to the ringtones app, which I already have a reviewing patch(bug 958470) and will fix both bug 958470 and this bug.

Apparently this issue should depends on bug 958470, and hopefully after clock use the ringtones pick activity to set the alram tones, this bug will be fixed automatically.
blocking-b2g: - → 1.4?
Depends on: 958470
Whiteboard: [AUDIO_COMPETING] → [AUDIO_COMPETING] [UX_TRIAGED]

Updated

4 years ago
Depends on: 959020

Updated

4 years ago
No longer depends on: 958470
Backlog - this doesn't hit the QC needs list & DSDS needs list, so this doesn't block.
blocking-b2g: 1.4? → backlog

Updated

4 years ago
Depends on: 958470

Comment 14

3 years ago
This issue  resolved-patch  is different with the   patch depends on 958470.

Comment 15

3 years ago
Clock rintone does not select from rintone App, and mozAudioChannelType is alarm. We need modify alarm_edit.js and time_panel.js file to resolve this issue.
Has this been fixed yet?
Flags: needinfo?(liang.xin)

Updated

3 years ago
Assignee: dkuo → nobody

Comment 17

3 years ago
(In reply to Patryk Adamczyk [:patryk] UX from comment #16)
> Has this been fixed yet?

This issue has been resolved
Flags: needinfo?(liang.xin)
(Assignee)

Updated

3 years ago
blocking-b2g: backlog → ---
tracking-b2g: --- → backlog
You need to log in before you can comment on or make changes to this bug.