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

Issue 719383 link

Starred by 1 user

Issue metadata

Status: Closed
Owner:
Closed: Nov 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Volume & brightness sliders are seen chopped in 'Reset this chrome device' screen.

Project Member Reported by jbanavatu@chromium.org, May 8 2017

Issue description

Chrome Version: 59.0.3071.39/9460.27.0 beta channel Peppy,Paine,Minnie,Jerry
OS: Chrome OS

What steps will reproduce the problem?
(1)Sign in to chrome >> Navigate to chrome://md-settings/reset page.
(2)Click on power wash option and click Restart button 
(3)Now in 'Reset this chrome device' screen, try decreasing or increasing either brightness or volume and Observe Volume & brightness sliders

Expected:Volume & brightness sliders should not be seen chopped  in 'Reset this chrome device' screen.
Actual: Instead,Volume & brightness sliders are seen chopped.

This is regression issue as it is working fine in 55.0.2883.105/8872.76.0 stable channel peppy.

Note: Issue is also seen in latest M60 #60.0.3089.0/9531.0.0 dev channel Peppy.

Attaching screen shots for reference.
 
Expected.jpg
2.5 MB View Download
Actual_Brightness.jpg
2.0 MB View Download
Actual_Volume.jpg
2.5 MB View Download

Comment 1 by r...@chromium.org, May 8 2017

Owner: steve...@chromium.org
Steven would this belong in MD Settings? If not, any idea who should own this?

Cc: steve...@chromium.org
Components: -UI UI>Shell>StatusArea
Owner: tdander...@chromium.org
Status: Assigned (was: Untriaged)
This is part of the system tray UI and unrelated to settings. -> tdanderson@ to triage


Cc: jamescook@chromium.org xiy...@chromium.org osh...@chromium.org
+xiyuan@, +oshima@, +jamescook@ - This looks like some weird interaction within views?

Cc: alemate@chromium.org minch@chromium.org tdander...@chromium.org jdufault@chromium.org
Labels: Needs-Bisect
Owner: jbanavatu@chromium.org
This could be due to any number of things:

* jdufault@'s recent work on the login/lock screens
* minch@'s recent work on the volume (but not brightness) slider
* alemate@'s new OOBE experience
* The MD-ification of the system menu and slider

The provided bisect range (m-55 to m-59/60) is far too large. Applying Needs-Bisect to request a revision-level bisect on the Chromium CL responsible (note that the offending CL is almost certainly in the Chromium, not Chromium OS, codebase).
Any webui should not be interacting with system tray. oobe/login/lock are a single views::WebView covering the entire screen.
I'll bet the "work area" for the screen is incorrect. No idea why that would happen, though.

I'm also unclear whether the status area (like the network icon and clock) should appear during these flows or not.

Re #6: I think that the status area should remain visible during these flows, as a user may wish to switch networks or enable an a11y feature.

Comment 8 by minch@chromium.org, May 8 2017

I tried to revert my cl about the volume slider.
This bug can still repros with my CL reverted locally.
https://codereview.chromium.org/2812223005/
Owner: tdander...@chromium.org
As we don not have bisect setup for chrome OS providing following info.Hope this is will be helpful.

Good:55.0.2883.105/8872.76.0
Bad:56.0.2924.121/9000.97.0

Note: Issue is also reproducible on 57.0.2987.154/9202.71.0 Beta and 58.0.3029.98/9334.64.0	Stable.

Thanks!
Cc: moh...@chromium.org abodenha@chromium.org
Owner: ----
Status: Available (was: Assigned)
I will be OOO for the next ~2 weeks or so. Marking as Available in case anyone else has cycles to dig into this a bit more.
./tools/binary-bisect.py should work for ChromeOS.

Comment 12 by ajha@chromium.org, Jun 14 2017

Cc: songsuk@chromium.org pucchakayala@chromium.org
Status: Untriaged (was: Available)
Back into UI > Shell triage queue.

Comment 14 by zalcorn@google.com, Jun 23 2017

Owner: tdander...@chromium.org
Status: Assigned (was: Untriaged)
tdanderson@ have you had a chance to look at this?
/triage
Components: UI>Shell>StartScreen
Owner: jdufault@chromium.org
Based on the large bisect range provided in #9, this regressed between revision #423769 (Oct 7 2016) and #433059 (Nov 18 2016). Skimming over the git revision log of chrome/browser/chromeos/login/ within this range, it looks as though jdufault@ is in the best position to suggest next steps here; I am not familiar with any of the login/lock type screens.

Based on the screenshots in #0 this could be a side effect of whenever the visuals of this screen were updated (perhaps due to a change in window bounds / positioning?)

As a side note, I can't reproduce this locally: with the command line flags --login-manager --login-screen=reset the sliders are not shown as clipped. This may be somehow device-specific.
Status: Closed (was: Assigned)
<triage> Cannot reproduce on 72.0.3593.0, please reopen if seen again. </triage>

Sign in to add a comment