Amy Lee [:amylee] UX Reporter
	Description • 11 years ago

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.

	Mishra
	Comment 1 • 11 years ago

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?

	Mishra
	Comment 2 • 11 years ago

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

	Mike Habicher [:mikeh] (high bugzilla latency)
	Comment 3 • 10 years ago

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

	Justin D'Arcangelo [:justindarc]
	Comment 4 • 10 years ago

(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!?

	Mike Habicher [:mikeh] (high bugzilla latency)
	Comment 5 • 10 years ago

Justin: yep, see bug 851823.

	Justin D'Arcangelo [:justindarc]
	Comment 6 • 10 years ago

Mike: Here's the requested logcat

	Mike Habicher [:mikeh] (high bugzilla latency)
	Comment 7 • 10 years ago

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.

	Justin D'Arcangelo [:justindarc]
	Comment 8 • 10 years ago

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.

	Justin D'Arcangelo [:justindarc]
	Comment 9 • 10 years ago

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

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

	:gerard-majax [PTO 01/08/2023-10/09/2023]
	Comment 10 • 10 years ago

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

	Justin D'Arcangelo [:justindarc]
	Comment 11 • 10 years ago

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

	Mike Habicher [:mikeh] (high bugzilla latency)
	Comment 12 • 10 years ago

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

	Mike Lien[:mlien]
	Comment 14 • 10 years ago

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

	Wilson Page [:wilsonpage]
	Comment 15 • 10 years ago

(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?

	Mike Lien[:mlien]
	Comment 16 • 10 years ago

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?

	Francis Lee [:frlee]
	Comment 17 • 10 years ago

hi youlong, 

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

	youlong.jiang
	Comment 18 • 10 years ago

Got it.

	Mike Lien[:mlien]
	Comment 19 • 10 years ago

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