Delay wallpaper load by 2 * average wallpaper load time.

chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc

 // Copyright (c) 2013 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.
 
 #include "chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h"
 
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
@@ skipping 405 matching lines @@
                  content::NotificationService::AllSources());
   registrar_.Add(this,
                  chrome::NOTIFICATION_AUTH_SUPPLIED,
                  content::NotificationService::AllSources());
   registrar_.Add(this,
                  chrome::NOTIFICATION_AUTH_CANCELLED,
                  content::NotificationService::AllSources());
 }
 
 SigninScreenHandler::~SigninScreenHandler() {
+  WallpaperManager::Get()->RemoveObserver(this);
   weak_factory_.InvalidateWeakPtrs();
   SystemKeyEventListener* key_event_listener =
       SystemKeyEventListener::GetInstance();
   if (key_event_listener)
     key_event_listener->RemoveCapsLockObserver(this);
   if (delegate_)
     delegate_->SetWebUIHandler(NULL);
   network_state_informer_->RemoveObserver(this);
 }
 
@@ skipping 110 matching lines @@
     UpdateUIState(UI_STATE_ACCOUNT_PICKER, &params);
   }
 }
 
 void SigninScreenHandler::ShowRetailModeLoginSpinner() {
   CallJS("showLoginSpinner");
 }
 
 void SigninScreenHandler::SetDelegate(SigninScreenHandlerDelegate* delegate) {
   delegate_ = delegate;
-  if (delegate_)
+  if (delegate_) {
     delegate_->SetWebUIHandler(this);
+    WallpaperManager::Get()->AddObserver(this);

[dzhioev (left Google), 2013/10/17 12:47:49]

Why do you add observer here? I see that |this| added as observer to network
state informer in constructor. I don't know which way is correct though.

[Alexander Alekseev, 2013/10/18 17:01:52]

Hmmm... I don't reember why, but there was a reason. Probably the reason has
gone. Moved to constructor.

+  }
 }
 
 void SigninScreenHandler::SetNativeWindowDelegate(
     NativeWindowDelegate* native_window_delegate) {
   native_window_delegate_ = native_window_delegate;
 }
 
 void SigninScreenHandler::OnNetworkReady() {
   MaybePreloadAuthExtension();
 }
@@ skipping 709 matching lines @@
 
 void SigninScreenHandler::HandleShutdownSystem() {
   ash::Shell::GetInstance()->lock_state_controller()->RequestShutdown();
 }
 
 void SigninScreenHandler::HandleLoadWallpaper(const std::string& email) {
   if (delegate_)
     delegate_->LoadWallpaper(email);
 }
 
+void SigninScreenHandler::OnWallpaperAnimationFinished(
+    const std::string& email) {
+  CallJS("login.AccountPickerScreen.onWallpaperLoaded", email);
+}
+
 void SigninScreenHandler::HandleRebootSystem() {
   chromeos::DBusThreadManager::Get()->GetPowerManagerClient()->RequestRestart();
 }
 
 void SigninScreenHandler::HandleRemoveUser(const std::string& email) {
   if (!delegate_)
     return;
   delegate_->RemoveUser(email);
   UpdateAddButtonStatus();
 }
@@ skipping 478 matching lines @@
   rvh->ExecuteJavascriptInWebFrame(
       ASCIIToUTF16("//iframe[@id='signin-frame']\n//iframe"),
       ASCIIToUTF16(code));
 
   // Test properties are cleared in HandleCompleteLogin because the form
   // submission might fail and login will not be attempted after reloading
   // if they are cleared here.
 }
 
 }  // namespace chromeos