Enable hide animation for login screen.

chrome/browser/chromeos/login/webui_login_display_host.cc

Index: chrome/browser/chromeos/login/webui_login_display_host.cc
diff --git a/chrome/browser/chromeos/login/webui_login_display_host.cc b/chrome/browser/chromeos/login/webui_login_display_host.cc
index 4fb1a9500c2f8e5f544cfd8987900ebc392a9b69..9fb1c9897323e95af833140d6cb534b983324471 100644
--- a/chrome/browser/chromeos/login/webui_login_display_host.cc
+++ b/chrome/browser/chromeos/login/webui_login_display_host.cc
@@ -6,6 +6,7 @@
 
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/time.h"
 #include "chrome/browser/chromeos/login/oobe_display.h"
 #include "chrome/browser/chromeos/login/webui_login_display.h"
 #include "chrome/browser/chromeos/login/webui_login_view.h"
@@ -17,6 +18,7 @@
 #if defined(USE_AURA)
 #include "ash/shell.h"
 #include "ash/shell_window_ids.h"
+#include "ash/wm/window_animations.h"
 #include "ui/aura/window.h"
 #endif
 
@@ -29,6 +31,9 @@ const char kLoginURL[] = "chrome://oobe/login";
 // URL which corresponds to the OOBE WebUI.
 const char kOobeURL[] = "chrome://oobe";
 
+// Duration of sign-in transition animation.
+const int kLoginFadeoutTransitionDurationMs = 700;
+
 }  // namespace
 
 // WebUILoginDisplayHost -------------------------------------------------------
@@ -41,7 +46,8 @@ WebUILoginDisplayHost::WebUILoginDisplayHost(const gfx::Rect& background_bounds)
 }
 
 WebUILoginDisplayHost::~WebUILoginDisplayHost() {
-  CloseWindow();
+  if (login_window_)
+    login_window_->Close();
 }
 
 // LoginDisplayHost implementation ---------------------------------------------
@@ -118,12 +124,15 @@ void WebUILoginDisplayHost::StartSignInScreen() {
   GetOobeUI()->ShowSigninScreen(webui_login_display_);
 }
 
-void WebUILoginDisplayHost::CloseWindow() {
+bvoid WebUILoginDisplayHost::OnBrowserCreated() {

[Nikita (slow), 2012/01/31 15:26:24]

void

[oshima, 2012/01/31 17:47:51]

Done.

+#if defined(USE_AURA)

[Nikita (slow), 2012/01/31 15:26:24]

Probably we don't need USE_AURA here?

[oshima, 2012/01/31 17:47:51]

We can do this on aura because window creation/deletion is synchronous on Aura.
It's async on native X/Gtk, and IIRC, we rely on WM to sync hiding and showing
browser window.

+  // Close lock window now so that the launched browser can receive focus.
   if (login_window_) {
     login_window_->Close();
     login_window_ = NULL;
     login_view_ = NULL;
   }
+#endif
 }
 
 void WebUILoginDisplayHost::LoadURL(const GURL& url) {
@@ -145,6 +154,12 @@ void WebUILoginDisplayHost::LoadURL(const GURL& url) {
     ash::Shell::GetInstance()->GetContainer(
         ash::internal::kShellWindowId_LockScreenContainer)->
         AddChild(login_window_->GetNativeView());
+    ash::SetWindowVisibilityAnimationDuration(
+        login_window_->GetNativeView(),
+        base::TimeDelta::FromMilliseconds(kLoginFadeoutTransitionDurationMs));
+    ash::SetWindowVisibilityAnimationPhase(
+        login_window_->GetNativeView(),
+        ash::ANIMATE_HIDE);
 #endif
 
     login_window_->SetContentsView(login_view_);