New issue
Advanced search Search tips

Issue 666836 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Dec 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Moving snapped window to external display makes for unhappiness.

Project Member Reported by abodenha@chromium.org, Nov 18 2016

Issue description

<b>Version: <Kenneth, what is the frequency?></b>
<b>OS: <please tell me it's not XP></b>

1. Connect large external monitor
2. go to chrome://settings-frame/display and position the external monitor above
the internal display
3. Snap a window to the right or left of the internal display
4. with the mouse grab to the top bar of the snapped window
5. Drag the window into the center of the external display


What is the expected result?
The window stays where you put it.

What happens instead?
When window is released on 30" display it jumps below the shelf on the 30"
monitor.

Please use labels and text to provide additional information.

 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cfb2ecbd4b3489acb4375ee9243176bd7ebd78ff

commit cfb2ecbd4b3489acb4375ee9243176bd7ebd78ff
Author: afakhry <afakhry@chromium.org>
Date: Thu Dec 01 22:49:57 2016

Fix dragging snapped window to another display landing the window in the wrong bounds

Dragging the snapped window to another display used to restore the wrong bounds
based on still-not-updated-yet root window during the drag. This resulted in the
window landing below the shelf and almost entirely not visible.

This CL fixes the issue and adds a test.

BUG= 666836 
TEST=ash_unittests --gtest_filter=ToplevelWindowEventHandlerTest.DragSnappedWindowToExternalDisplay

Review-Url: https://codereview.chromium.org/2538293002
Cr-Commit-Position: refs/heads/master@{#435747}

[modify] https://crrev.com/cfb2ecbd4b3489acb4375ee9243176bd7ebd78ff/ash/common/wm/default_state.cc
[modify] https://crrev.com/cfb2ecbd4b3489acb4375ee9243176bd7ebd78ff/ash/common/wm/workspace/workspace_window_resizer.cc
[modify] https://crrev.com/cfb2ecbd4b3489acb4375ee9243176bd7ebd78ff/ash/wm/toplevel_window_event_handler_unittest.cc

Status: Fixed (was: Started)

Comment 4 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 5 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59
Status: Verified (was: Fixed)
9532.0.0, 60.0.3092.0

Sign in to add a comment