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

Hema Please review.

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

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.

Over to Gaia.

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

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.

diego, please update with your investigation thanks hema

Still need investigation

ETA 03/21/2014

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

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?

Review the patch again :)

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.

Landed in camera-new-features: https://github.com/mozilla-b2g/gaia/commit/20e3128a651cd28970e53469a1d61f8cf754c42a

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

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

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.

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

sorry, reopening it until we land on 1.4

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

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

Found Test Case: https://moztrap.mozilla.org/manage/case/1858/ STR needs to added to verify this bug to the existing test case.

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/