New issue
Advanced search Search tips

Issue 609501 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Every maximized window resizes when switching profiles with multiple monitors.

Project Member Reported by flackr@chromium.org, May 5 2016

Issue description

Version: 50.0.2661.91 (Official Build) (64-bit)
OS: ChromeOS

Looks like the bug in  issue 385992  was only fixed for a single monitor. This is the same thing but for multiple monitors.

What steps will reproduce the problem?
(1) Attach a second monitor.
(2) Log in to multiple profiles.
(3) Maximize a window or two.
(4) Switch profiles
(5) Switch back.

What is the expected output?
None of the windows should resize. The attached resize-log.html should log no resizes.

What do you see instead?
Instead, every maximized window is resized each time you switch profiles. This greatly increases the jank when switching profiles, especially when you have a few maximized windows open on each profile.

Please use labels and text to provide additional information.

 
resize-log.html
316 bytes View Download
Owner: x...@chromium.org
Status: Assigned (was: Untriaged)

Comment 2 by x...@chromium.org, Jun 6 2017

Cc: x...@chromium.org
Owner: sammiequon@chromium.org
sammiequon@, could you try to see if it's still can be repro'ed?
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 16 2017

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

commit 1e73e98a5aa5a31e4e4bdc17c1a945a631cedb76
Author: sammiequon <sammiequon@chromium.org>
Date: Fri Jun 16 17:04:35 2017

wm: Do not resize when switching users if window is already maximized.

Extends  crbug.com/385992  to work when more than one displays are attached. More details in bug and  crbug.com/385992 .

TEST=unit_tests --gtest_filter="MultiUserWindowManagerChromeOSTest.TestBlackBarCover"
BUG= 609501 

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

[modify] https://crrev.com/1e73e98a5aa5a31e4e4bdc17c1a945a631cedb76/chrome/browser/ui/ash/multi_user/multi_user_window_manager_chromeos_unittest.cc
[modify] https://crrev.com/1e73e98a5aa5a31e4e4bdc17c1a945a631cedb76/chrome/browser/ui/ash/multi_user/user_switch_animator_chromeos.cc

Status: Fixed (was: Assigned)

Comment 5 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment