Issue metadata
Sign in to add a comment
|
Regression: Weird and chopped Powerwash screen is seen |
||||||||||||||||||||||
Issue descriptionChrome Version:68.0.3405.0/10621.0.0 dev channel Daisy,Candy,Reks OS:Chrome OS What steps will reproduce the problem? (1)In Sign out screen >>Press (Ctrl+Alt+shift+R) for Powerwash overlay and observe Actual: weird and chopped Powerwash overlay is seen Expected: Instead, it shouldn't get chopped This is a Regression issue as same is working in 68.0.3397.0/10595.0.0 dev Note: Issue is not seen in Windows,Linux OS Attached the screencast for reference.. @wzang : Please confirm the issue.
,
Apr 27 2018
Seems to be from the version string. I thought there was a bug for that already, but I was unable to locate it.
,
Apr 27 2018
,
May 1 2018
The version string shown in the screenshot is html, so I think we should just hide it when views-based login is enabled. But the views-based version string is not shown on the login screen yet (it's a separate bug). That's because the version info is not fetched for the login screen. Where's the best place to move the code in [1] so that it can be shared between login/lock? [1] https://cs.chromium.org/chromium/src/chrome/browser/chromeos/login/lock/views_screen_locker.cc?type=cs&sq=package:chromium&l=95
,
May 2 2018
Re#4: I think it's ok for views login screen to get the version info separately because views login and views lock screen don't share the same code path in the back end, and we can do that in LoginDisplayHostMojo for views login screen.
,
May 16 2018
Issue 839310 has been merged into this issue.
,
Jun 7 2018
,
Jun 27 2018
I will try to reproduce after the powerwash keyboard shortcut is landed.
,
Jul 17
Reassigning to qnnguyen@ since he is already doing work with powerwash screen
,
Jul 31
I don't think this should be a problem anymore. Seems fixed for me. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by wzang@chromium.org
, Apr 27 2018Owner: jdufault@chromium.org