919418 - [B2G][Helix][Audio][tianhuajian]The camera can record incall ringtone sound

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect, P2)

Description

[lecky]

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; aff-kingsoft-ciba)

Steps to reproduce:

A is FFOS phone , B is another phone
1. A open the camera app
2. A take video recording by using camera app
3. B call A
4. A connect the call
5. A disconnect the call
6. A open the video player
7. Play the video file recorded by camera app



Actual results:

7. In the video file end , it  has ringtone sound


Expected results:

7. The video file has no ringtone sound

Comment 1

[lecky]

【Test Count*】：5
【Found Count*】：5
【Gaia commit ID*】: V1.1HD
【Gecko commit ID*】:  1.1.0 HD
【Log*】：
【Network environment】：
【Resume operation】：

Comment 2

[lecky]

I find in audio hal layer, the audio first change audio device, then stopRecording , and it is the reason the camera record has ringtone sound.

Line 2658: 09-22 10:12:38.621: WARN/AudioHardwareMSM76XXA(169): rpc_snd_set_device(58, 1, 0)
Line 2826: 09-22 10:12:39.351: INFO/QCameraHWI(169): stopRecording: X

Comment 3

[Wayne Chang [:wchang]]

Hi Bruce, is this a bug or do you want to backlog this?

Comment 4

[Bruce Huang [:bhuang] (inactive)]

Edge case, backlogging.

Comment 5

[lecky]

Hi Beatriz,
This issue Mozilla think is a edge case and backlogging it.
We hope you can check whether this case is a issue, 
and if we do not fix this issue on our build, Can you support this result?
Thanks. 
(In reply to Bruce Huang (:bhuang) from comment #4)
> Edge case, backlogging.

Comment 6

[Beatriz Rodríguez [:brg]]

We agree this is an edge case but we think it is a good improvement in current behaviour. Bruce, can we nominate it to 1.3?

Comment 7

[Bruce Huang [:bhuang] (inactive)]

Nom for 1.3

Comment 8

[Marco Chen [:mchen]]

Set the category back to Gaia:Camera, because the stopping recording is not controlled by AudioChannel policy. It is controlled by Camera app itself by detecting the visibility change event.

Comment 9

[lecky]

Hi Bruce,
Do you mean this issue will be resolved on V1.3?
Thanks

(In reply to Bruce Huang (:bhuang) from comment #7)
> Nom for 1.3

Comment 10

[Bruce Huang [:bhuang] (inactive)]

No, it means we will need to consider it for 1.3 depending on resourcing and priorities.

Comment 11

[Hema Koka [:hema]]

Bruce, can you triage this with your teams

Comment 12

[Marco Chen [:mchen]]

Hi,

Please refer to the link as below. This bug should be the same root cause with that one.
https://bugzilla.mozilla.org/show_bug.cgi?id=908525#c51

Comment 13

[Ivan Tsay (:ITsay)]

Nominate it to 1.5 because it needs big effort for fixing the issue.

Comment 14

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Alive, is this related to bug 927862 like bug 908525? Can we fix bug 927862 in 1.5 time frame?

Comment 15

[(inactive after 6/18) Alive Kuo [:alive]]

Yes, attention-window is in 1.5 time frame.

Comment 16

[Ivan Tsay (:ITsay)]

Backlog this one and plan it in the future sprint.

Comment 17

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Is this bug not fixed by bug 927862? qawanted for master branch check.

Comment 18

[Cody Roesch [:croesch]]

Tested with Shallow Flash on 319mb using Engineering builds

This bug DOES exist in the latest Flame 2.2 KK build. Ringtone is heard for the last 5 or so seconds of the video before it stops.

Repro Rate: 3/3

Environmental Variables:
Device: Flame 2.2 KK
BuildID: 20141110052655
Gaia: e02facadb0bc3fe32227b52b31ca47822f7f4322
Gecko: 1a09297482e1
Version: 36.0a1 (2.2) 
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

Comment 19

[BMO Automation]

Firefox OS is not being worked on