Fix black background when locking with fullscreen window:

ash/wm/session_state_controller_impl2.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ASH_WM_SESSION_STATE_CONTROLLER_IMPL2_H_
 #define ASH_WM_SESSION_STATE_CONTROLLER_IMPL2_H_
 
 #include "ash/ash_export.h"
 #include "ash/shell_observer.h"
 #include "ash/wm/session_state_animator.h"
@@ skipping 24 matching lines @@
 class ASH_EXPORT SessionStateControllerImpl2 : public SessionStateController {
  public:
 
   // Helper class used by tests to access internal state.
   class ASH_EXPORT TestApi {
    public:
     explicit TestApi(SessionStateControllerImpl2* controller);
 
     virtual ~TestApi();
 
-    bool lock_timer_is_running() const {
-      return controller_->lock_timer_.IsRunning();
-    }
     bool lock_fail_timer_is_running() const {
       return controller_->lock_fail_timer_.IsRunning();
     }
     bool lock_to_shutdown_timer_is_running() const {
       return controller_->lock_to_shutdown_timer_.IsRunning();
     }
     bool shutdown_timer_is_running() const {
       return controller_->pre_shutdown_timer_.IsRunning();
     }
     bool real_shutdown_timer_is_running() const {
       return controller_->real_shutdown_timer_.IsRunning();
     }
+    bool is_animating_lock() const {
+      return controller_->animating_lock_;
+    }
+    bool is_lock_undoable() const {
+      return controller_->undoable_lock_animation_;
+    }
 
-    void trigger_lock_timeout() {
-      controller_->OnLockTimeout();
-      controller_->lock_timer_.Stop();
-    }
     void trigger_lock_fail_timeout() {
       controller_->OnLockFailTimeout();
       controller_->lock_fail_timer_.Stop();
     }
     void trigger_lock_to_shutdown_timeout() {
       controller_->OnLockToShutdownTimeout();
       controller_->lock_to_shutdown_timer_.Stop();
     }
     void trigger_shutdown_timeout() {
       controller_->OnPreShutdownAnimationTimeout();
       controller_->pre_shutdown_timer_.Stop();
     }
     void trigger_real_shutdown_timeout() {
       controller_->OnRealShutdownTimeout();
       controller_->real_shutdown_timer_.Stop();
     }
    private:
     SessionStateControllerImpl2* controller_;  // not owned
 
     DISALLOW_COPY_AND_ASSIGN(TestApi);
   };
 
+  struct UnlockedStateProperties {

[sky, 2012/12/12 20:35:56]

Is there a reason you aren't using a bool directly?

[Denis Kuznetsov (DE-MUC), 2012/12/13 17:59:50]

Yes, it seems that there would be similar parameters for launcher state later,
and I want to have them groupped together.

+    bool background_is_hidden;
+  };
+
   SessionStateControllerImpl2();
   virtual ~SessionStateControllerImpl2();
 
   // RootWindowObserver override:
   virtual void OnRootWindowHostCloseRequested(
      const aura::RootWindow* root) OVERRIDE;
 
   // ShellObserver overrides:
   virtual void OnLoginStateChanged(user::LoginStatus status) OVERRIDE;
   virtual void OnAppTerminating() OVERRIDE;
@@ skipping 19 matching lines @@
 
   virtual void OnLockScreenHide(base::Closure& callback) OVERRIDE;
   virtual void SetLockScreenDisplayedCallback(base::Closure& callback) OVERRIDE;
 
  protected:
   friend class test::SessionStateControllerImpl2Test;
 
  private:
   void RequestShutdownImpl();
 
-  // Starts lock timer.
-  void StartLockTimer();
-
-  // Requests that the screen be locked and starts |lock_fail_timer_|.
-  void OnLockTimeout();
-
   // Reverts the pre-lock animation, reports the error.
   void OnLockFailTimeout();
 
   // Starts timer for gap between lock and shutdown.
   void StartLockToShutdownTimer();
 
   // Calls StartShutdownAnimation().
   void OnLockToShutdownTimeout();
 
   // Starts timer for undoable shutdown animation.
   void StartPreShutdownAnimationTimer();
 
   // Calls RequestShutdownImpl();
   void OnPreShutdownAnimationTimeout();
 
   // Starts timer for final shutdown animation.
-  void StartRealShutdownTimer();
+  // If |with_animation_time| is true, it will also include time of "fade to
+  // white" shutdown animation.
+  void StartRealShutdownTimer(bool with_animation_time);
 
   // Requests that the machine be shut down.
   void OnRealShutdownTimeout();
 
+  // Starts shutdown animation that can be cancelled and starts pre-shutdown
+  // timer.
+  void StartCancellableShutdownAnimation();
+
+  // Starts non-cancellable animation and starts real shutdown timer that
+  // includes animation time.
+  void StartShutdownAnimationImpl();
+
   // Triggers late animations on the lock screen.
   void OnLockScreenAnimationFinished();
 
+  // Methods for initiating and checking different phases of
+  // lock/unlock animations:
+  // Lock phase one : windows lift (can be undone in some cases)
+  // Lock phase two : lock screen raises
+  // Unlock phase one : lock screen lowers
+  // Unlock phase two : windows drop down
+
+  void StartImmediateLockAnimationPhaseOne();
+  void StartUndoableLockAnimationPhaseOne();
+  void UndoLockAnimationPhaseOne();
+  void StartLockAnimationPhaseTwo();
+  // This method calls |callback| when animation completes.
+  void StartUnlockAnimationPhaseOne(base::Closure &callback);
+  void StartUnlockAnimationPhaseTwo();
+
+  // These methods are called when corresponding animation completes.
+  void LockAnimationUndone();
+  void LockAnimationPhaseOneCompleted();
+  void LockAnimationPhaseTwoCompleted();
+  void UnlockAnimationPhaseTwoCompleted();
+
+  // Stores properties of UI that have to be temporary modified while locking.
+  void StoreUnlockedProperties();
+  void RestoreUnlockedProperties();
+
+  // Fades in background layer with |speed| if it was hidden in unlocked state.
+  void AnimateBackgroundAppearanceIfNecessary(
+      ash::internal::SessionStateAnimator::AnimationSpeed speed,
+      ui::LayerAnimationObserver* observer);
+
+  // Fades out background layer with |speed| if it was hidden in unlocked state.
+  void AnimateBackgroundHidingIfNecessary(
+      ash::internal::SessionStateAnimator::AnimationSpeed speed,
+      ui::LayerAnimationObserver* observer);
+
   // The current login status, or original login status from before we locked.
   user::LoginStatus login_status_;
 
   // Current lock status.
   bool system_is_locked_;
 
   // Are we in the process of shutting the machine down?
   bool shutting_down_;
 
   // Indicates whether controller should proceed to (cancellable) shutdown after
   // locking.
   bool shutdown_after_lock_;
 
-  // Started when the user first presses the power button while in a
-  // logged-in-as-a-non-guest-user, unlocked state.  When it fires, we lock the
-  // screen.
-  base::OneShotTimer<SessionStateControllerImpl2> lock_timer_;
+  // Indicates that controller displays lock animation.
+  bool animating_lock_;
+
+  // Indicates that lock animation can be undone.
+  bool undoable_lock_animation_;

[sky, 2012/12/12 20:35:56]

can_stop_lock_animation_.

[Denis Kuznetsov (DE-MUC), 2012/12/13 17:59:50]

Done.

+
+  scoped_ptr<UnlockedStateProperties> unlocked_properties_;

[sky, 2012/12/12 20:35:56]

Why the scoped_ptr instead of by value?

[Denis Kuznetsov (DE-MUC), 2012/12/13 17:59:50]

One reason is that I want to have just another layer of defence if things go
wrong, like, make sure that we never would use value from 2 locks ago -
properties are reset on every unlock.

 
   // Started when we request that the screen be locked.  When it fires, we
   // assume that our request got dropped.
   base::OneShotTimer<SessionStateControllerImpl2> lock_fail_timer_;
 
   // Started when the screen is locked while the power button is held.  Adds a
   // delay between the appearance of the lock screen and the beginning of the
   // pre-shutdown animation.
   base::OneShotTimer<SessionStateControllerImpl2> lock_to_shutdown_timer_;
 
   // Started when we begin displaying the pre-shutdown animation.  When it
   // fires, we start the shutdown animation and get ready to request shutdown.
   base::OneShotTimer<SessionStateControllerImpl2> pre_shutdown_timer_;
 
   // Started when we display the shutdown animation.  When it fires, we actually
   // request shutdown.  Gives the animation time to complete before Chrome, X,
   // etc. are shut down.
   base::OneShotTimer<SessionStateControllerImpl2> real_shutdown_timer_;
 
   base::Closure lock_screen_displayed_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(SessionStateControllerImpl2);
 };
 
 }  // namespace ash
 
 #endif  // ASH_WM_SESSION_STATE_CONTROLLER_IMPL2_H_