New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 819083 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

CCA: Delay from timer is applied to recording mode

Project Member Reported by shenghao@chromium.org, Mar 6 2018

Issue description

Steps to reproduce:
1. Open CCA. Enter photo mode.
2. Turn on timer.
3. Switch to recording mode.
4. Start recording.

Observed that the recording does not start immediately. Turning on the timer in photo mode also affects the recording mode.

Expected behavior:
Timer should not be applied in recording mode.

 
Cc: shenghao@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 7 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/apps/camera/+/1bb7a80954335713bed3b58dfb272dc372e9da14

commit 1bb7a80954335713bed3b58dfb272dc372e9da14
Author: Yuli Huang <yuli@chromium.org>
Date: Wed Mar 07 06:39:32 2018

Fix timer incorrectly active for video recording.

BUG= chromium:819083 
TEST=Tested manually on elm.

Change-Id: I6d6b88d2e2f15cbc6c9b21a221184bcf49cdf532
Reviewed-on: https://chromium-review.googlesource.com/950742
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>
Tested-by: yuli <yuli@chromium.org>

[modify] https://crrev.com/1bb7a80954335713bed3b58dfb272dc372e9da14/src/js/views/camera.js

Comment 3 by y...@chromium.org, Mar 7 2018

Status: Fixed (was: Assigned)
Please verify it on the upcoming camera app canary.
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 15

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1bb7a80954335713bed3b58dfb272dc372e9da14

commit 1bb7a80954335713bed3b58dfb272dc372e9da14
Author: Yuli Huang <yuli@chromium.org>
Date: Wed Mar 07 06:39:32 2018

Fix timer incorrectly active for video recording.

BUG= chromium:819083 
TEST=Tested manually on elm.

Change-Id: I6d6b88d2e2f15cbc6c9b21a221184bcf49cdf532
Reviewed-on: https://chromium-review.googlesource.com/950742
Reviewed-by: Sheng-hao Tsao <shenghao@google.com>
Tested-by: yuli <yuli@chromium.org>

[modify] https://crrev.com/1bb7a80954335713bed3b58dfb272dc372e9da14/src/js/views/camera.js

Sign in to add a comment