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

Issue 891961 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
OOO (till 1/28)
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

[CCA] Caroline: camera preview screen show wrong rotations in portrait/tablet mode

Project Member Reported by allendam@chromium.org, Oct 4

Issue description

Chrome Version: 11120.0.0, 71.0.3567.0 (dev) channel
Camera vers: CCA 5.2.2
Devices: Caroline PVT

Step:
(1) Switch device into (Portrait/Tablet) mode
(2) Launch CCA 5.2.2 in portrait mode (90 degrees) (either Front/Back facing)
Observe: see attached files

Expected result:
CCA app should open in portrait mode without any issue

Actual result:
Camera preview screen show wrong rotation 

- maybe related to bug: https://bugs.chromium.org/p/chromium/issues/detail?id=888826

How frequently does this problem reproduce? (100%)  

Feedback ID: https://listnr.corp.google.com/product/208/report/85701644997
 
Cc: jcliang@chromium.org shik@chromium.org
shik@, it might be related to #888826; would you please help to take a look? Thanks!
The screenshot/repro seems not related to #888826. I'll check it.
Also reproducible on EVE PVT with build 11125.0.0, 71.0.3567.0 (dev)channel
Feedback ID: https://listnr.corp.google.com/product/208/report/85703580216
Owner: shenghao@chromium.org
I couldn't repro on Caroline (M71 build) and sent the device to shenghao@.

#3: The screenshot looks like the device was in landscape mode but not in portrait mode?
Can't reproduce on 11120.0.0 caroline-arcnext.
allendam@, was the screen rotation mode auto-rotate?
Owner: allendam@chromium.org
Owner: shenghao@chromium.org
Still reproducible on Caroline MP with build 11150.0.0, 71.0.3575.1 (dev) channel (CCA 5.2.4)(screen rotation mode set Auto-rotate ON)
Feedback ID: https://listnr.corp.google.com/product/208/report/85719123441
shenghao@,
#7 screenshot looks like 888826? However, we still couldn't get a repro on these devices.

allendam@,
Does this issue still have 100% repro rate on Caroline?
Still reproducible on Caroline with build M72 - 11171.0.0 - 72.0.3582.0. (100% repro)
Feedback ID: https://listnr.corp.google.com/product/208/report/85731632666
Stillreproducible on EVE PVT with build 11151.6.0, 71.0.3578.13 (dev)channel
Hi Allen,
Does the bug reproduce for CCA 5.2.5 (the public version)?
You probably need to disable ARC container on the device and then click the camera app icon to trigger the public version.
Still reproducible on Caroline, Robo360 with build 11151.16.0, 71.0.3578.27 (beta) channel.  (CCA 5.2.5 with ARC++ diabled)  

Feedback ID: 
Caroline - https://listnr.corp.google.com/product/208/report/85752635671
Robo360 - https://listnr.corp.google.com/product/208/report/85752638695
Cc: xutan@chromium.org skuhne@chromium.org
Testing on 11197.0.0, I discovered that the problem only exists on caroline, but not on caroline-arcnext. 

https://cs.chromium.org/chromium/src/media/capture/video/chromeos/display_rotation_observer.cc?type=cs&g=0&l=84

ScreenObserverDelegate::SendDisplayRotationOnCaptureThread() is not called. Therefore all the frames are treated as if screen_rotation=0. 

Stefan and Xu Tan, do you know any recent changes that could have led to this?

Cc: vsu...@google.com
 Issue 888826  has been merged into this issue.
Hmm... It's weird that N and P has different behavior. I can't think of anything on top of my head.
Cc: hywu@chromium.org allendam@chromium.org
 Issue 914626  has been merged into this issue.
Owner: skuhne@chromium.org
This bug is still reproducible on 72/73 builds.
Stefan, could you triage to correct people about why ScreenObserverDelegate::SendDisplayRotationOnCaptureThread() is not called?

We have another repro:
https://buganizer.corp.google.com/issues/122048506#comment6
The regression seems to happen after 11140.0.0.
Status: Assigned (was: Untriaged)
This issue has an owner, a component and a priority, but is still listed as untriaged or unconfirmed. By definition, this bug is triaged. Changing status to "assigned". Please reach out to me if you disagree with how I've done this.

Sign in to add a comment