984730 - Camcorder starts with flash light by default

Status: VERIFIED FIXED

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

Description

[bhargavg1]

when ever camcorder is started for the first time on the v1.4 for KK, see that it starts with the "SetFlashMode" as "torch" and enables flash light till user disables that.

can we make the default value as "none" instead of torch

Comment 1

[Preeti Raghunath(:Preeti)]

Hema

Please review.

Comment 2

[Hema Koka [:hema]]

Mike, 

If this is quick, please take care of it. If not, we will need to have Joe get help from those working on kit-kat in Taipei.

Thanks
Hema

Comment 3

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

I've confirmed that (on the Nexus 4, at least) the flash defaults to 'off'. When we bring the camera up in picture mode, the Camera app sets the flash mode to 'auto', and when we switch to video mode, it sets the flash mode to 'torch'.

Swapping the order of the flashModesVideo items in app.js doesn't seem to affect whether or not the flash is on, so whatever it setting this is doing it intentionally.

Comment 4

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

Over to Gaia.

Comment 5

[Hema Koka [:hema]]

jcheng - anyone available to pick this up from folks working on kitkat/camera...

Comment 6

[Diego Marcos [:dmarcos]]

I don't think this is KK specific. In the branch I can see that the default value for flash is on but it should be auto. This is why video recording starts in torch mode. Investigating the reason.

Comment 7

[Hema Koka [:hema]]

diego, please update with your investigation

thanks
hema

Comment 8

[Diego Marcos [:dmarcos]]

Still need investigation

Comment 9

[Diego Marcos [:dmarcos]]

ETA 03/21/2014

Comment 10

[Diego Marcos [:dmarcos]]

Attachment: Pull Request on camera new features

Comment 11

[Diego Marcos [:dmarcos]]

Should we create a flag default for the options in the configuration file? Now the default value is the last option of the list.

Comment 12

[Justin D'Arcangelo [:justindarc]]

Diego: See comments on PR. I'm pretty sure we have the ability to specify the default in the config/app.js file using the `selected` key. Can you try setting that to see if that works instead of re-ordering?

Comment 13

[Diego Marcos [:dmarcos]]

Review the patch again :)

Comment 14

[Inder]

Hema: this is a one line patch. Can we please land it in 1.4 as well. This is marked as a blocker by test teams.

Comment 15

[Diego Marcos [:dmarcos]]

Landed in camera-new-features:

https://github.com/mozilla-b2g/gaia/commit/20e3128a651cd28970e53469a1d61f8cf754c42a

Comment 16

[Inder]

Diego -- can you please land it on gaia/v1.4 as well?

Comment 17

[Jason Smith [:jsmith]]

This isn't landed on 1.4 in the mainline branch yet.

Comment 18

[Diego Marcos [:dmarcos]]

Please keep this closed. It's going to be landed in 1.4 when we merge camerea-new-features after fixing all the blockers. Partner requirement.

Comment 19

[Jason Smith [:jsmith]]

(In reply to Diego Marcos [:dmarcos] from comment #18)
> Please keep this closed. It's going to be landed in 1.4 when we merge
> camerea-new-features after fixing all the blockers. Partner requirement.

A landing process has nothing to do with a partner requirement - this is about being able to track what's currently present in the mainline branches. The camera-new-features branch isn't considered a mainline branch, so bugs fixed there aren't considered resolved in a mainline branch. What's even worse here is deciding to close this early will cause confusion to others filing bugs, as they'll see the bug closed, even though the bug is still present in the mainline branch. This can also screw up tree managers, as they could potentially think the patch is landed on trunk, when in reality it's not.

Feel free to use the whiteboards to track landings in the custom branch. However, if the problem is present in the mainline Gaia branches, then an issue isn't considered resolved if it doesn't land there.

Comment 20

[Hema Koka [:hema]]

sorry, reopening it until we land on 1.4

Comment 21

[David Flanagan [:djf]]

Bulk edit for camera bugs.

If earlier comments do not show how this bug landed to master, it probably landed as part of https://github.com/mozilla-b2g/gaia/pull/17599 which merged the camera-new-features branch into master.

This bug was uplifted from master to v1.4 as part of https://github.com/mozilla-b2g/gaia/commit/a8190d08e61316a86bba572ba8d894d081a20530

Comment 22

[sarsenyev]

The bug is fixed on 1.4 and master build
The camcorder flash is off by default

1.4 Environmental Variables:
Device: Leo 1.4 MOZ
BuildID: 20140411000202
Gaia: 6c50349f41d40ba175ea0fc0c2c2cbd739ba7170
Gecko: 28b419f0e857
Version: 30.0a2
Firmware Version: v10d

1.5 Environmental Variables:
Device: Leo 1.5 MOZ
BuildID: 20140407040202
Gaia: f1a98bfaa3ab2480945bd7018831fd56c61cdc24
Gecko: 5405d6f4e3c6
Version: 31.0a1
Firmware Version: v10d

Comment 23

[Yeojin Chung [:YeojinC]]

Found Test Case: https://moztrap.mozilla.org/manage/case/1858/

STR needs to added to verify this bug to the existing test case.

Comment 24

[KTucker [:KTucker][Inactive 3/4/2016]]

Test case updated to check that flash is off by default for video and set to auto by default for camera:

https://moztrap.mozilla.org/manage/case/1858/