[CCA] Caroline: camera preview screen show wrong rotations in portrait/tablet mode |
|||||||
Issue descriptionChrome 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
,
Oct 4
The screenshot/repro seems not related to #888826. I'll check it.
,
Oct 5
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
,
Oct 5
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?
,
Oct 12
Can't reproduce on 11120.0.0 caroline-arcnext. allendam@, was the screen rotation mode auto-rotate?
,
Oct 12
,
Oct 12
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
,
Oct 13
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?
,
Oct 18
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
,
Oct 19
Stillreproducible on EVE PVT with build 11151.6.0, 71.0.3578.13 (dev)channel
,
Oct 22
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.
,
Oct 30
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
,
Oct 30
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?
,
Oct 30
,
Oct 30
Hmm... It's weird that N and P has different behavior. I can't think of anything on top of my head.
,
Dec 18
,
Dec 18
This bug is still reproducible on 72/73 builds. Stefan, could you triage to correct people about why ScreenObserverDelegate::SendDisplayRotationOnCaptureThread() is not called?
,
Jan 2
We have another repro: https://buganizer.corp.google.com/issues/122048506#comment6 The regression seems to happen after 11140.0.0.
,
Jan 11
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 |
|||||||
Comment 1 by y...@chromium.org
, Oct 4