New issue
Advanced search Search tips

Issue 915797 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Dec 18
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

[CCA 5.3] Nocturne: not able to switch to "Back" facing camera mode

Project Member Reported by allendam@chromium.org, Dec 17

Issue description

Chrome Version:11316.22.0, 72.0.3626.15(dev) channel
Android: 9
Camera vers: CCA 5.3 canery
Devices: Nocturne DVT main 4 (has 2 cameras)

Steps:
1. Launch CCA 5.3 in (desktop/landscape/small/full screen mode)
2. Switch camera to "BACK" facing

Observe below: 

Expected result:
able to switch to "back" facing camera mode 

Actual result:
Not able to switch to "BACK" facing camera mode

How frequently does this problem reproduce? (100%)  

Feedback ID:
 
Also reproducible with Nautilus CCA 5.3 canery.  
Feedback ID: https://listnr.corp.google.com/report/85852445855
Cc: -wtlee@chromium.org -inishan@chromium.org jcliang@chromium.org shik@chromium.org
shenghao@, Would this be a device issue?
Owner: shenghao@chromium.org
Owner: y...@chromium.org
The console shows error logs when I click on switch button. Please see the attached screenshot.
I think it's a CCA issue.
image.png
58.1 KB View Download
Got it; thanks much for catching this. I'll prepare a fix.
Status: Assigned (was: Untriaged)
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 18

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

commit 806ed857fd1b6963b7f3e570cdf0e8a16abcdaaa
Author: Yuli Huang <yuli@chromium.org>
Date: Tue Dec 18 09:08:34 2018

Fix naming conflicts in options.js.

BUG= chromium:915797 
TEST=Tested by clicking switch-mode/device buttons.

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

[modify] https://crrev.com/806ed857fd1b6963b7f3e570cdf0e8a16abcdaaa/src/css/main.css
[modify] https://crrev.com/806ed857fd1b6963b7f3e570cdf0e8a16abcdaaa/src/views/main.html
[modify] https://crrev.com/806ed857fd1b6963b7f3e570cdf0e8a16abcdaaa/src/js/views/camera/options.js

Status: Fixed (was: Assigned)
Please verify the fix in the next canary update. Thanks.
Not reproducible: Tested on Nocturne, Dru, Nautilus with CCA 5.3.1 canary. Front/Back camera is working fine.
Status: Verified (was: Fixed)
Project Member

Comment 11 by bugdroid1@chromium.org, Jan 15

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

commit 806ed857fd1b6963b7f3e570cdf0e8a16abcdaaa
Author: Yuli Huang <yuli@chromium.org>
Date: Tue Dec 18 09:08:34 2018

Fix naming conflicts in options.js.

BUG= chromium:915797 
TEST=Tested by clicking switch-mode/device buttons.

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

[modify] https://crrev.com/806ed857fd1b6963b7f3e570cdf0e8a16abcdaaa/src/css/main.css
[modify] https://crrev.com/806ed857fd1b6963b7f3e570cdf0e8a16abcdaaa/src/js/views/camera/options.js
[modify] https://crrev.com/806ed857fd1b6963b7f3e570cdf0e8a16abcdaaa/src/views/main.html

Sign in to add a comment