Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(605)

Unified Diff: ash/root_window_controller.cc

Issue 10909008: Improve existing lock transition - remove black splash. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | ash/wm/power_button_controller.h » ('j') | ash/wm/power_button_controller_unittest.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/root_window_controller.cc
diff --git a/ash/root_window_controller.cc b/ash/root_window_controller.cc
index c6c047fa7dd6b2a2c5c5cc1af353e595ef320215..28132eac10b13fbfd6a540f396fa8b9d1d328c96 100644
--- a/ash/root_window_controller.cc
+++ b/ash/root_window_controller.cc
@@ -132,12 +132,31 @@ void SetUsesScreenCoordinates(aura::Window* container) {
void CreateContainersInRootWindow(aura::RootWindow* root_window) {
// These containers are just used by PowerButtonController to animate groups
// of containers simultaneously without messing up the current transformations
- // on those containers. These are direct children of the root window; all of
+ // on those containers. These are direct children of the root window; all of
// the other containers are their children.
+ // Desktop and lock screen background containers are not part of the
+ // lock animation so they are not included in those animate groups.
+ // When screen is locked desktop background is moved to lock screen background
+ // container (moved back on unlock). We want to make sure that there's an
+ // opaque layer occluding the non-lock-screen layers.
+
+ aura::Window* desktop_background_containers = CreateContainer(
+ internal::kShellWindowId_DesktopBackgroundContainer,
+ "DesktopBackgroundContainer",
+ root_window);
+ SetChildWindowVisibilityChangesAnimated(desktop_background_containers);
+
aura::Window* non_lock_screen_containers = CreateContainer(
internal::kShellWindowId_NonLockScreenContainersContainer,
"NonLockScreenContainersContainer",
root_window);
+
+ aura::Window* lock_background_containers = CreateContainer(
+ internal::kShellWindowId_LockScreenBackgroundContainer,
+ "LockScreenBackgroundContainer",
+ root_window);
+ SetChildWindowVisibilityChangesAnimated(lock_background_containers);
+
aura::Window* lock_screen_containers = CreateContainer(
internal::kShellWindowId_LockScreenContainersContainer,
"LockScreenContainersContainer",
@@ -154,12 +173,6 @@ void CreateContainersInRootWindow(aura::RootWindow* root_window) {
CreateContainer(internal::kShellWindowId_SystemBackgroundContainer,
"SystemBackgroundContainer", non_lock_screen_containers);
- aura::Window* desktop_background_containers = CreateContainer(
- internal::kShellWindowId_DesktopBackgroundContainer,
- "DesktopBackgroundContainer",
- non_lock_screen_containers);
- SetChildWindowVisibilityChangesAnimated(desktop_background_containers);
-
aura::Window* default_container = CreateContainer(
internal::kShellWindowId_DefaultContainer,
"DefaultContainer",
@@ -211,13 +224,6 @@ void CreateContainersInRootWindow(aura::RootWindow* root_window) {
non_lock_screen_containers);
SetUsesScreenCoordinates(input_method_container);
- aura::Window* lock_background_containers = CreateContainer(
- internal::kShellWindowId_LockScreenBackgroundContainer,
- "LockScreenBackgroundContainer",
- lock_screen_containers);
-
- SetChildWindowVisibilityChangesAnimated(lock_background_containers);
-
// TODO(beng): Figure out if we can make this use
// SystemModalContainerEventFilter instead of stops_event_propagation.
aura::Window* lock_container = CreateContainer(
« no previous file with comments | « no previous file | ash/wm/power_button_controller.h » ('j') | ash/wm/power_button_controller_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698