Closed Bug 796286 Opened 12 years ago Closed 12 years ago

[Camera] No shutter sound

Categories

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

defect

Tracking

(blocking-basecamp:-)

VERIFIED FIXED
blocking-basecamp -

People

(Reporter: ghtobz, Assigned: daleharvey)

References

Details

(Keywords: reproducible, Whiteboard: [label:camera][label:needsVISUALinput])

Attachments

(3 files)

[GitHub issue by timdream on 2012-09-18T08:23:54Z, https://github.com/mozilla-b2g/gaia/issues/4851] STR: 1. Take a picture with the Camera app Expected: 1. Pic taken with a stutter sound coming from the phone Actual: 1. No sound Note: - Shutter sound is a legal requirement in some jurisdictions, although the bill died in US: http://thomas.loc.gov/cgi-bin/bdquery/z?d111:HR00414:@@@L&summ2=m& - To meet the legal requirement, such shutter sound cannot be muted. This is a situation similar related to #4847, as discussed on dev-b2g.
[GitHub comment by benfrancis on 2012-09-18T13:10:34Z] Hmm, is it confirmed that this is a legal requirement in the jurisdictions v1 is launching in? I'd find it annoying if I couldn't turn it off (incidentally this is a pretty dumb piece of legislation given that there's no sound when you're shooting a video!)
[GitHub comment by timdream on 2012-09-18T16:15:24Z] I am not sure about the situation in Brazil.
[GitHub comment by autonome on 2012-09-24T15:48:39Z] Not blocking until confirmed there's a legal requirement.
[GitHub comment by daleharvey on 2012-09-26T02:53:09Z] @patrykdesign This isnt a blocker as mentioned above, I did implement it though however the sound file provided didnt work, if I swap out the sound file for the unlock sound it works fine Both files play fine in firefox when opened directly, it may be some fairly strange file format differences, not all that experience with audio formats in firefox though
[GitHub comment by patrykdesign on 2012-09-28T19:14:57Z] @daleharvey thanks, I'll swap it with the right fine when I get it from the vendor.
Shutter sound is depend on the ability for the web content to sound as a different stream type.
Depends on: 795237
[mass adding reproducible keyword for any open Gaia bug with the word "STR:" in comments]
Keywords: reproducible
Tim, who could make this happen?
(In reply to Patryk Adamczyk [:patryk] UX from comment #8) > Tim, who could make this happen? Make what happen? To put the sound into Gaia you can just ask @daleharvey again. To make the sound always audible even if the phone is muted we need bug 795237.
Priority: -- → P2
Attached video Use this! (deleted) —
Assignee: padamczyk → dale
Blocks: 797140
Attachment #678563 - Flags: review?(mhabicher)
Attachment #678563 - Flags: review?(mhabicher) → review+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Verified sound occurs when taking a picture, but not when the sound is muted. Tested on the 11/26 unagi build.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached file Play shutter sound even when muted (deleted) —
Attachment #686074 - Flags: review?(etienne)
(In reply to Dale Harvey (:daleharvey) from comment #14) > Created attachment 686074 [details] > Play shutter sound even when muted Could you help to try mozAudioChannelType = 'publicnotification' ? 1. The alarm type still have chance to be set as muted. 2. The publicnotification is designed to play a sound with never muted. Thanks for your testing in first.
Attachment #686074 - Flags: review?(etienne) → review+
publicnotification works, thanks for the headsup mchen merged in: https://github.com/mozilla-b2g/gaia/pull/6680
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Device: Unagi Build ID: 20121231070201 Defect does not reproduce using the Unagi device. Sound is very faint but you can hear it.
(In reply to austincmiller from comment #17) > Device: Unagi > Build ID: 20121231070201 > Defect does not reproduce using the Unagi device. Sound is very faint but > you can hear it. Please make sure if you are saying this bug is confirmed fixed that you mark it as verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: