1578945 - Remove old audio competing mechanism in AudioChannelService

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, task, P2)

Description

[Alastor Wu [:alwu]]

We have implemented new audio focus management in bug1565689 which would allow us to have one tab playing audio at the same time no matter we're using e10s or Fission.

Therefore, we can remove the old pref dom.audiochannel.audioCompeting and dom.audiochannel.audioCompeting.allAgents, which are only work on non-e10s mode.

In addition, the audio competing is only used by default on Fennec, but Fennec is now no longer following the latest m-c, it's using ESR68. So even if our new audio focus management is not on by default on Android, it won't affect current behavior on Fennec. We can 100% sure that we won't need those code for old audio competing anymore.

Comment 1

[Alastor Wu [:alwu]]

Attachment: Bug 1578945 - part1 : remove handling audio focus related code in AudioChannelService.

We have implemented new audio focus management in bug1565689 which would allow us to have one tab playing audio at the same time no matter we're using e10s or Fission.

Therefore, we can remove the old pref dom.audiochannel.audioCompeting and dom.audiochannel.audioCompeting.allAgents, which are only work on non-e10s mode.

In addition, the audio competing is only used by default on Fennec, but Fennec is now no longer following the latest m-c, it's using ESR68. So even if our new audio focus management is not on by default on Android, it won't affect current behavior on Fennec. We can 100% sure that we won't need those code for old audio competing anymore.

Comment 2

[Alastor Wu [:alwu]]

Attachment: Bug 1578945 - part2 : remove old audio competing related tests.

Comment 3

[Alastor Wu [:alwu]]

Sorry, I just notice that we can't remove this right now, because these audio competing code is still running on Fenix. Therefore, instead of removing them immeidiately, I would start to enable new audio focus managment on Fenix Nightly only, and see how it works. If it works well, then we will turn it on Fenix Release, and then replace the old mechanism steps by steps.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:alwu, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 5

[Alastor Wu [:alwu]]

I will wait more time to see if enabling new audio competing on Fenix causes any regression or not, in order to decide whether we would like to land these two patches.

Comment 6

[Alastor Wu [:alwu]]

Attachment: Bug 1578945 - part3 : enable new audio competing mechanism for all versions on Android.

We've enabled new the audio competing mechanism on Android Nightly two months ago, and we didn't get any bug report about it, so it's time to enable that for all versions on Android.

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1582802

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fab28f89bb44
part1 : remove handling audio focus related code in AudioChannelService. r=baku,sebastian
https://hg.mozilla.org/integration/autoland/rev/9f2c46528648
part2 : remove old audio competing related tests. r=baku
https://hg.mozilla.org/integration/autoland/rev/6309c331099d
part3 : enable new audio competing mechanism for all versions on Android. r=sebastian

Comment 8

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/fab28f89bb44
https://hg.mozilla.org/mozilla-central/rev/9f2c46528648
https://hg.mozilla.org/mozilla-central/rev/6309c331099d