Closed Bug 1657091 Opened 4 years ago Closed 3 years ago

SoundCloud is not paused/does not pause other audio on Android

Categories

(Core :: Audio/Video: Playback, defect, P3)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: padenot, Unassigned)

Details

This is probably because SoundCloud uses an AudioContext for output. We should probably pause the HTMLMediaElement iff it's not playing a MediaStream, something like this.

Alastor, do you have thoughts about this?

Flags: needinfo?(alwu)

What do you mean not pausing other audio on Android? You mean other tab playing media? Could you describe your testing steps?

Flags: needinfo?(alwu) → needinfo?(padenot)

STR:

  • Play audio on the device (I'm using the Spotify Android app, but anything works)
  • Open a track on SoundCloud in Firefox for Android, tap the central play icon to start playback

Expected:

  • Spotify is paused, SoundCloud in Firefox plays

Actual:

  • Spotify is not paused, SoundCloud in Firefox plays
Flags: needinfo?(padenot)

If the page is youtube instead of SoundCloud, is the Spotify app being paused?

I am going to make this P3 and S4. Feel free to bump the priority or severity if needed.

Severity: -- → S4
Priority: -- → P3

(In reply to C.M.Chang[:chunmin] from comment #3)

If the page is youtube instead of SoundCloud, is the Spotify app being paused?

Yes. Youtube uses an HTMLMediaElement.

Severity: S4 → S2
Priority: P3 → P1

NI myself to take a look later.

Flags: needinfo?(alwu)

On Fenix, requesting platform audio focus is done by Android component [1] which is not related with gecko's implementation.

[1] https://github.com/mozilla-mobile/android-components/tree/bc4695121f01d276aadc6473a379d541a1569e29/components/feature/media/src/main/java/mozilla/components/feature/media/focus


Hi, Sebastian,
Is this a known issue for the audio focus component? I wonder whether it would help if Fenix uses the new GV MediaSession API [2] to implement the audio focus?
Thank you.

[2] https://bugzilla.mozilla.org/show_bug.cgi?id=1623715

Flags: needinfo?(alwu) → needinfo?(s.kaspari)
QA Whiteboard: qa-not-actionable

Delegating to Roger.

Flags: needinfo?(s.kaspari) → needinfo?(royang)

I will look into this when I have a chance

Flags: needinfo?(royang)
Severity: S2 → S4
Priority: P1 → P3

This works well now, I just tried it.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.