New issue
Advanced search Search tips

Issue 834773 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Screen freezes when rotated

Project Member Reported by osh...@chromium.org, Apr 19 2018

Issue description

chrome: at commit position #551809

Chrome believes it's still running, but no updates on screen.

It was working the day before, so must be very recent regression.
Running bisect.
 

Comment 1 by osh...@chromium.org, Apr 19 2018

Owner: sadrul@chromium.org
bisected to https://crrev.com/c/1014568

sadrul@, this will make tablet device (Dru) unusable, so please look into it asap.
let me know if you just want to revert it for now.

Comment 2 by sadrul@chromium.org, Apr 19 2018

Status: Started (was: Assigned)

Comment 3 by osh...@chromium.org, Apr 19 2018

Cc: wutao@chromium.org
Components: UI>Shell

Comment 4 by sadrul@chromium.org, Apr 19 2018

https://chromium-review.googlesource.com/c/chromium/src/+/1020085 is the fix I currently have. Briefly, the problem is that SetBoundsInPixels() on WindowTreeHost is called without the size actually changing, or without assigning a valid LocalSurfaceId. It may make sense to move that change so that it takes care of similar issues in other places.

Comment 5 by wutao@chromium.org, Apr 19 2018

I tested the patch on EVE and it works. The animation smoothness value is consistent with the UMA metrics.

Comment 6 by osh...@chromium.org, Apr 27 2018

Labels: -Pri-1 Pri-0
Bumping to p0  because this is causing various graphics glitches on the device. If you can't fix this soon, I'll revert it for now.

Comment 7 by osh...@chromium.org, Apr 27 2018

Bumping to p0  because this is causing various graphics glitches on the device. If you can't fix this soon, I'll revert it for now.

Comment 8 by sadrul@chromium.org, Apr 27 2018

I will disable some of the tests in mash-browser-tests so the fix in https://chromium-review.googlesource.com/c/chromium/src/+/1020085 can land.
Labels: -Pri-0 Pri-1
Since mus is no longer turned on by default, this is no longer an issue, right?
If this is not used by default, you can lower to P2.

Sign in to add a comment