cros: HTML fixes needed to reinitializing an existing lock WebUI.

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 <stddef.h>
 
 #include <algorithm>
 #include <vector>
@@ skipping 882 matching lines @@
   // Forces a reload for Gaia screen on hiding error message.
   if (IsGaiaVisible() || IsGaiaHiddenByError())
     ReloadGaia(reason == NetworkError::ERROR_REASON_NETWORK_STATE_CHANGED);
 }
 
 void SigninScreenHandler::ReloadGaia(bool force_reload) {
   gaia_screen_handler_->ReloadGaia(force_reload);
 }
 
 void SigninScreenHandler::Initialize() {
+  // delegate_ is null when we are preloading the lock screen. There may be no

[xiyuan, 2016/12/09 22:47:21]

nit: delegate_ -> |delegate_|

[jdufault, 2016/12/12 19:52:02]

Done.

+  // primary user in a test.
+  if (user_manager::UserManager::IsInitialized() &&
+      user_manager::UserManager::Get()->GetPrimaryUser()) {
+    chromeos::PinStorage* pin_storage =

[Alexander Alekseev, 2016/12/10 01:26:49]

I assume that you are only looaking at active user because this is a lock screen
and at the time the lock screen is preloaded only one user is active, aren't
you? Probably we need a comment here why only one user is checked for PIN.

[jdufault, 2016/12/12 19:52:02]

I've changed this to iterate over every user.

+        chromeos::PinStorageFactory::GetForUser(
+            user_manager::UserManager::Get()->GetActiveUser());
+    if (pin_storage && pin_storage->IsPinAuthenticationAvailable())
+      CallJS("cr.ui.Oobe.preloadPin");
+  }
+
   if (delegate_ && show_on_init_) {
     show_on_init_ = false;
     ShowImpl();
   }
 }
 
 gfx::NativeWindow SigninScreenHandler::GetNativeWindow() {
   if (native_window_delegate_)
     return native_window_delegate_->GetNativeWindow();
   return nullptr;
@@ skipping 620 matching lines @@
 }
 
 void SigninScreenHandler::OnFeedbackFinished() {
   CallJS("login.UnrecoverableCryptohomeErrorScreen.resumeAfterFeedbackUI");
 
   // Recreate user's cryptohome after the feedback is attempted.
   HandleResyncUserData();
 }
 
 }  // namespace chromeos