New issue
Advanced search Search tips

Issue 922223 link

Starred by 2 users

Issue metadata

Status: Untriaged
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Arc App window can't be restored to its original bounds after being snapped

Project Member Reported by x...@chromium.org, Jan 15

Issue description

Chrome Version: 73.0.3672.0
Arc: 5228037
OS: Chrome

What steps will reproduce the problem?
(1) Open a N-resizable Arc app, for example, Youtube, Gmail in normal mode
(2) Press ALT+[ or ALT+] to snap the window to left or right
(3) Press ALT+[ or ALT+] again

What is the expected result?
The Arc app window should restore to its position and bounds, just like what Chrome browser windows or apps do.

What happens instead?
The window is not restored.


 
Cc: zork@chromium.org
Owner: amusbach@chromium.org
I think the reason is for Arc app windows, we do not have bounds restore logic yet. Thus when the second ALT+[ or Alt+] is pressed, ClientControlledState::HandleCompoundEvents() calls BaseState::CycleSnap() calls WindowState::Restore(), it only restores the window state from SNAPPED to NORMAL, but the bounds is not restored. 

We may need to add the similar logic as what DefaultState does to ClientControlledState to set and clear the restore bounds when different WM Events come in. 

amusbach@, can you take a look?

Sign in to add a comment