Commit 4aa1c73f authored by Sammie Quon's avatar Sammie Quon Committed by Commit Bot

cros: Fix issue with window being black if minimized before tablet mode.

Issue was:
1) Open two windows in clamshell, minimize both.
2) Enter tablet.
3) Activate both windows, second window will be a black screen.

It seems minimizing in clamshell mode sets opacity to 0. So force opacity
to 1 before activate to ensure visiblity.

Test: manual
Bug: 920463
Change-Id: I3858ca4ac771d23879678f8705b73a281df0b8ac
Reviewed-on: https://chromium-review.googlesource.com/c/1406243Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622171}
parent d29eec20
...@@ -698,7 +698,10 @@ bool HomeLauncherGestureHandler::SetUpWindows(Mode mode, aura::Window* window) { ...@@ -698,7 +698,10 @@ bool HomeLauncherGestureHandler::SetUpWindows(Mode mode, aura::Window* window) {
if (window && !windows.empty() && windows[0] != window && if (window && !windows.empty() && windows[0] != window &&
windows[0]->IsVisible()) { windows[0]->IsVisible()) {
// Do not run slide down animation for the |window| if another active // Do not run slide down animation for the |window| if another active
// window in mru list exists. // window in mru list exists. Windows minimized in clamshell mode may
// have opacity of 0, so set them to 1 to ensure visibility.
if (wm::GetWindowState(window)->IsMinimized())
window->layer()->SetOpacity(1.f);
window1_.reset(); window1_.reset();
return false; return false;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment