1013313 - [Flame][Camera] Two sounds play when turning video record mode on/off

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Vendcom, defect, P1)

Description

[Amy Lee [:amylee] UX]

There are two sounds that play when you hit video record on/off. The correct sound is the 2nd one that is played when you hit start video record.

Steps to reproduce: 

1. Open camera
2. Switch to video record mode
3. Hit record button. You hear two sounds being played one after the other.

Tested on Flame device.

Comment 1

[Mishra]

Device    Nexus 4
BuildID   20140516110816
Version   32.0a1
Git Commit info 2014-05-05 16:59:23 d33e4f15

I tested this on Nexus 4 but this issue is not present there.
Infact I could not listen to any sound. 
On which build was the above issue reported?

Comment 2

[Mishra]

I enabled the Setting->Sound->Video Recording and it only plays one sound

Comment 3

[Mike Habicher [:mikeh] (high bugzilla latency)]

Not sure if the Sora issue is related, but I posted some questions in bug 975349 comment 3 and 4.

Comment 4

[Justin D'Arcangelo [:justindarc]]

(In reply to Mishra from comment #2)
> I enabled the Setting->Sound->Video Recording and it only plays one sound

I just enabled this setting also and I am definitely able to reproduce it on the Flame, but not the Nexus 4. On the Flame, there is a sound that is played when recording starts/stops that seems to be independent (and different) than the start/stop sounds for the Camera app.

Mike: I just saw your comments in Bug 975349 -- is it actually possible for the Camera driver to emit sounds on its own!?

Comment 5

[Mike Habicher [:mikeh] (high bugzilla latency)]

Justin: yep, see bug 851823.

Comment 6

[Justin D'Arcangelo [:justindarc]]

Attachment: logcat of mystery start/stop sounds

Mike: Here's the requested logcat

Comment 7

[Mike Habicher [:mikeh] (high bugzilla latency)]

It certainly looks like the camera library is initiating a sound:

05-21 12:51:56.956   283   619 E QCamera2HWI: [KPI Perf] static int qcamera::QCamera2HardwareInterface::start_recording(camera_device*): E PROFILE_START_RECORDING
  ...
05-21 12:51:56.966   283   617 D ALSADevice: route: devices 0x2 in mode 0
  ...
05-21 12:51:56.966   283   617 D ALSADevice: open: handle 0xb900a4b0, format 0x2
05-21 12:51:56.966   283   617 D ALSADevice: Music case
05-21 12:51:56.966   283   617 D ALSADevice: Device value returned is hw:0,0
05-21 12:51:56.966   283   617 D ALSADevice: flags 0, devName hw:0,0
05-21 12:51:56.966   283   617 D ALSADevice: handle->format: 0x2
05-21 12:51:56.966   283   617 D ALSADevice: setHardwareParams: reqBuffSize 2048 channels 2 sampleRate 48000
05-21 12:51:56.966   283   617 D ALSADevice: setHardwareParams: buffer_size 16384, period_size 2048, period_cnt 8

...although there's nothing definitive. I'll take a look at the Flame code to see if there's anything obvious in there.

Comment 8

[Justin D'Arcangelo [:justindarc]]

There's an interesting idea to resolve this in Bug 858469 that is to simply remove the AOSP system sound files from the device that are being played on recording start/stop.

Comment 9

[Justin D'Arcangelo [:justindarc]]

The path to the sound file on the Flame that needs to be removed is:

/system/media/audio/ui/VideoRecord.ogg

Comment 10

[:gerard-majax]

Just FTR, bug 851823 was the same on Nexus S.

Comment 11

[Justin D'Arcangelo [:justindarc]]

root@msm8610:/ # ls -l /system/media/audio/ui/VideoRecord.ogg
-rw-r--r-- root     root         6483 2014-05-19 21:39 VideoRecord.ogg

Comment 12

[Mike Habicher [:mikeh] (high bugzilla latency)]

/system/media is not part of our build, so this is POVB.

Comment 14

[Mike Lien[:mlien]]

verify with Flame KK base image v162-3, it's fine

Comment 15

[Wilson Page [:wilsonpage]]

(In reply to Mike Lien[:mlien] from comment #14)
> verify with Flame KK base image v162-3, it's fine

Can we close this then?

Comment 16

[Mike Lien[:mlien]]

It's my mistake, I misunderstood this issue for.
Flame KK base image still has "VideoRecording.ogg"
If we don't want to hear this voice, we may ask partner to remove from their side

Hi Francis, could you help on this?

Comment 17

[Francis Lee [:frlee]]

hi youlong, 

could you please help to remove /system/media/audio/ui/VideoRecord.ogg for next KK build?

Comment 18

[youlong.jiang]

Got it.

Comment 19

[Mike Lien[:mlien]]

verify with Flame KK base image v184 and VideoRecord.ogg is already removed