Commit 338c0e1b authored by Sammie Quon's avatar Sammie Quon Committed by Commit Bot

overview: Fix issue with blur not sometimes not working.

Issue happens when animation blur to start and no animation to end, or
vice versa. Slide animation skips when it thinks the animation is already
shown or not shown. For direct blur settings when there is no animation,
set the slide animation's state as well, so it has knowledge when blur
is set not using slide animation.

Test: manual
Bug: 852160
Change-Id: Ib4a0be9606e65d10f7cfdc542019f870a5ecfe58
Reviewed-on: https://chromium-review.googlesource.com/1098462Reviewed-by: default avatarMitsuru Oshima <oshima@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#566888}
parent 4140260d
...@@ -168,6 +168,7 @@ class WindowSelectorController::OverviewBlurController ...@@ -168,6 +168,7 @@ class WindowSelectorController::OverviewBlurController
DCHECK(window_selector); DCHECK(window_selector);
for (aura::Window* root : Shell::Get()->GetAllRootWindows()) { for (aura::Window* root : Shell::Get()->GetAllRootWindows()) {
if (!window_selector->ShouldAnimateWallpaper(root)) { if (!window_selector->ShouldAnimateWallpaper(root)) {
animation_.Reset(should_blur ? 1.0 : 0.0);
ApplyBlur(root, value); ApplyBlur(root, value);
} else { } else {
root->AddObserver(this); root->AddObserver(this);
......
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