Closed Bug 1191730 Opened 9 years ago Closed 6 years ago

[Aries][Browser]Audio will pause if you lock screen while playing a video from youtube.

Categories

(Firefox OS Graveyard :: Gaia::Browser, defect, P2)

ARM
Gonk (Firefox OS)
defect

Tracking

(tracking-b2g:backlog, b2g-v2.2 unaffected, b2g-master affected)

RESOLVED WONTFIX
tracking-b2g backlog
Tracking Status
b2g-v2.2 --- unaffected
b2g-master --- affected

People

(Reporter: huayu.li, Unassigned)

References

Details

(Keywords: regression, Whiteboard: [2.5-aries-test-run-1])

Attachments

(3 files)

Attached file logcat3.txt
[1.Description]:
[AriesKK v2.5][FlameKK v2.5]Play a video from youtube, and then press power key to lock screen, the audio will be paused.
Found at:17:10
See attachment:logcat3.txt, Aries2.3gp

[2.Testing Steps]: 
1. Open the Browser app. 
2. Go to a streaming media provider (such as YouTube). 
3. Play any video. 
4. While the video is streaming, lock the screen by pressing the power button. Pay close attention to the audio of the video.

[3.Expected Result]: 
The audio is still playing smoothly without any gaps in the audio. 

[4.Actual Result]: 
4.Audio paused.

[5.Reproduction build]: 
Device: Aries KK 2.5(Affected)
Build ID               20150803195455
Gaia Revision          dbacf8364f4505d021b7d8fb9cabea325004dbcc
Gaia Date              2015-08-03 16:38:49
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/abc56d57f6e1
Gecko Version          42.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20150803.191800
Firmware Date          Mon Aug  3 19:18:08 UTC 2015
Bootloader             s1

Device: Flame KK 2.2(Unaffected)
Build ID               20150805032505
Gaia Revision          f8b119ac30e97df991c97682ac4d4f9ca22e1793
Gaia Date              2015-07-31 13:20:55
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/0c7a85251e10
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150805.065842
Firmware Date          Wed Aug  5 06:58:54 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device:Flame KK 2.5(Affected)
Build ID               20150805150207
Gaia Revision          581de383687dc441a878d2c91a0167c6ec688fef
Gaia Date              2015-08-05 01:48:40
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/b12a261ee32e
Gecko Version          42.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150805.195702
Firmware Date          Wed Aug  5 19:57:12 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

[6.Reproduction Frequency]: 
Always Recurrence,5/5

[7.TCID]: 
13289
Keywords: regression
Attached video Aries2.3gp
QA Contact: mshuman
This issue appears to be caused by:
Bug 1113086 - Implement AudioChannel API into BrowserElement

Mozilla-Inbound Regression Window:

Last Working 
Environmental Variables:
Device: Flame 2.5
BuildID: 20150710163851
Gaia: e4b63559eba364892867eb381c3002d6518e5d6a
Gecko: 07bcf36f5ab2
Version: 42.0a1 (2.5)
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0

First Broken 
Environmental Variables:
Device: Flame 2.5
BuildID: 20150710170452
Gaia: e4b63559eba364892867eb381c3002d6518e5d6a
Gecko: 675ea719b91c
Version: 42.0a1 (2.5)
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0

Last Working gaia / First Broken gecko - Issue DOES reproduce
Gaia: e4b63559eba364892867eb381c3002d6518e5d6a
Gecko: 675ea719b91c

First Broken gaia / Last Working gecko - Issue does NOT reproduce
Gaia: e4b63559eba364892867eb381c3002d6518e5d6a
Gecko: 07bcf36f5ab2

Mozilla Inbound Pushlog: 
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=07bcf36f5ab2&tochange=675ea719b91c
Blocks: 1113086
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Gerry, can you take a look at this please? This might have been caused by the landing for bug 1113086. The author of this landing is currently on PTO.
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker) → needinfo?(gchang)
Hi Andrea,
Can you take a look at this bug? It seems to be caused by bug 1113086.
Flags: needinfo?(gchang) → needinfo?(amarchesini)
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][COM=Private Browsing]
Flags: needinfo?(amarchesini) → needinfo?(alwu)
Check it later.
Hi, all,
I think that we didn't allow playback youtube video in background, other video as well.
As I known, we only allow music app playback to do that.
Thanks.
Flags: needinfo?(alwu)
Is behavior changed? Does any UX spec mention this? The issue doesn't happen in 2.2.
Flags: needinfo?(alwu)
Hi, Gerry,
UX sound doesn't mention about that.
However, I test the same situation in the Android Nexus5, the video would be paused.
Maybe we need the UX people to verify this bug.

--

Hi, Bobby,
Could you know who is responsible for the audio UX?
Thanks.
Flags: needinfo?(alwu) → needinfo?(bchien)
Tori, could you help to comment on this? Thanks.
Flags: needinfo?(bchien) → needinfo?(tchen)
From UX perspective I think we should be careful on this issue. There are two type use cases: one is users leave the app/lock the screen because they really want to leave the content consuming(video/audio); the other is users want to keep consuming content(audio as background music).

We can enable background music, but for users who want to entirely close the website, it can be annoying(Image that everytime you close the tab it still playing music and you can not find the pause button because it is so small). 

Proposal 1: Provide visual hint on the screen(such as which page is playing music) and an easy way to pause/end music on the lock screen.

Proposal 2: Only enable background music for some popular music/video websites(youtube, etc).

Or we can do proposal 1 + proposal 2. Thanks.
Flags: needinfo?(tchen) → needinfo?(bchien)
Sorry for the typos, paragraph 2 should be ->
We can enable background music, but for users who want to entirely close the website, it can be annoying(Imagine that everytime you leave the page or lock the screen but it still playing music, and you can not find the pause button because it is small and hidden inside the page).
Hi Ben and Dale, what do you think about Tori's proposal?
Flags: needinfo?(dale)
Flags: needinfo?(bfrancis)
Flags: needinfo?(bchien)
This bug has been filed multiple times in the past, sometimes people saying they expect sound to continue playing when they switch to a different window/turn off the screen, and some people complaining when the sound stops playing. I think the behaviour did change at some point but I don't fully understand what changed.

Personally I often use my Firefox OS device to listen to podcasts and it's annoying when I can't switch to another window and do something at the same time because the audio stops (using PlayerFM). There are times when I want YouTube to keep playing when I turn off the screen, but also times when audio is playing and I can't figure out how to stop it!

I like proposal 1, I don't like proposal 2. But really this issue doesn't belong in the Browser component and you should ask someone in the Media team. I'm not sure what the standards say, exactly how our audio channels work, how much is currently under the control of content, and how much is under the control of the system.
Flags: needinfo?(bfrancis)
I think we need a specific definition about "whether the background web content can playback audio/video".

In present design, we use the "content" type to allow the app playing audio on the background, ex. Music app. The app's manifest need to declare the audio channel type first, then it can have the permission to use that type. 

However, in web content, because it's not an app, it doesn't have the extra permission to use the "content" type. The web content can only use the "normal" type, which would be paused when it turns to background. The bug1084464 have some discussions about this issue, but now we don't have the WIP patch.

The reason why we can playback the "video" on the background before is that we did some hacks in Gecko, and the audio policy logic was managed by Gecko. Now the audio policy is controlled by the system app of Gaia, so these hack have also been removed.
[Tracking Requested - why for this release]:
Agreed that this is fairly out of scope of the browser and needs dedicated media thoughts, I think having the task manager / statusbar give indication of audio in tabs seems like a fairly clear win, I dont have enough information about the problem to say whether we should need content to specifically opt in to background music and how they would do that.
Flags: needinfo?(dale)
I found Chrome for Android has similar feature: they provide indicator in status bar and notification center when video is playing
Priority: -- → P2
See Also: → 1190434
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: