cros: Preload and reuse the lock screen across lock sessions

chrome/browser/chromeos/login/lock/webui_screen_locker.cc

 // Copyright 2014 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/chromeos/login/lock/webui_screen_locker.h"
 
 #include "ash/common/wm_shell.h"
 #include "ash/shell.h"
 #include "ash/system/chromeos/power/power_event_observer.h"
 #include "base/command_line.h"
+#include "base/feature_list.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/browser_shutdown.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/chromeos/accessibility/accessibility_util.h"
 #include "chrome/browser/chromeos/login/helper.h"
 #include "chrome/browser/chromeos/login/lock/screen_locker.h"
+#include "chrome/browser/chromeos/login/ui/shared_web_view.h"
+#include "chrome/browser/chromeos/login/ui/shared_web_view_factory.h"
 #include "chrome/browser/chromeos/login/ui/webui_login_display.h"
+#include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/ui/webui/chromeos/login/oobe_ui.h"
 #include "chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h"
+#include "chrome/common/chrome_features.h"
+#include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
+#include "chrome/grit/generated_resources.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
+#include "components/prefs/pref_service.h"
 #include "components/user_manager/user.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/render_widget_host_view.h"
+#include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_ui.h"
 #include "ui/aura/client/capture_client.h"
 #include "ui/aura/window_event_dispatcher.h"
+#include "ui/base/l10n/l10n_util.h"
 #include "ui/base/x/x11_util.h"
 #include "ui/display/display.h"
 #include "ui/display/screen.h"
 #include "ui/keyboard/keyboard_controller.h"
 #include "ui/keyboard/keyboard_util.h"
 #include "ui/views/controls/webview/webview.h"
 
 namespace {
 
 // URL which corresponds to the login WebUI.
 const char kLoginURL[] = "chrome://oobe/lock";
 
 // Disables virtual keyboard overscroll. Login UI will scroll user pods
 // into view on JS side when virtual keyboard is shown.
 void DisableKeyboardOverscroll() {
   keyboard::SetKeyboardOverscrollOverride(
       keyboard::KEYBOARD_OVERSCROLL_OVERRIDE_DISABLED);
 }
 
 void ResetKeyboardOverscrollOverride() {
   keyboard::SetKeyboardOverscrollOverride(
       keyboard::KEYBOARD_OVERSCROLL_OVERRIDE_NONE);
 }
 
+chromeos::WebUILoginView::WebViewSettings BuildSettings() {
+  chromeos::WebUILoginView::WebViewSettings settings;
+  if (chromeos::WebUIScreenLocker::ShouldShareLockScreen()) {
+    settings.preloaded_url = GURL(kLoginURL);
+    settings.web_view_title =
+        l10n_util::GetStringUTF16(IDS_LOCK_SCREEN_TASK_MANAGER_NAME);
+  }
+  return settings;
+}
+
 }  // namespace
 
 namespace chromeos {
 
+// static
+bool WebUIScreenLocker::ShouldShareLockScreen() {
+  Profile* profile = ProfileHelper::Get()->GetProfileByUser(
+      user_manager::UserManager::Get()->GetActiveUser());
+
+  // Note that |profile| can be null in tests.
+  return base::FeatureList::IsEnabled(features::kSharedLockScreen) && profile &&
+         profile->GetPrefs()->GetBoolean(prefs::kEnableAutoScreenLock);

[xiyuan, 2016/12/09 22:59:55]

Why do we need to associate with kEnableAutoScreenLock ?

[jdufault, 2016/12/12 20:08:27]

It is the primary heuristic to try and figure out if the user actively locks
their screen. If set to false then no lock screen will appear after a device
goes to sleep.

Maybe an alternative would be an unsycned pref recording the last time the user
manually locked their screen. If it has been longer than a week, don't preload?

[xiyuan, 2016/12/12 20:33:19]

How about using the following:

https://cs.chromium.org/chromium/src/chrome/browser/chromeos/login/users/chro...

[xiyuan, 2016/12/12 20:43:36]

On a second thought, the problem we are trying to solve is the timeout when lock
on suspending. So what you have makes sense now. :) Let's comment that clearly.

[jdufault, 2016/12/12 21:06:27]

I've added a comment.

+}
+
+// static
+void WebUIScreenLocker::Preload() {
+  DCHECK(ShouldShareLockScreen());
+  VLOG(1) << "Preloading lock screen";
+
+  SharedWebView* shared_web_view =
+      SharedWebViewFactory::GetForProfile(ProfileHelper::GetSigninProfile());
+
+  views::WebView* web_view;
+  if (!shared_web_view->Get(GURL(kLoginURL), &web_view)) {
+    web_view->LoadInitialURL(GURL(kLoginURL));
+    InitializeWebView(
+        web_view, l10n_util::GetStringUTF16(IDS_LOCK_SCREEN_TASK_MANAGER_NAME));
+  }
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // WebUIScreenLocker implementation.
 
 WebUIScreenLocker::WebUIScreenLocker(ScreenLocker* screen_locker)
-    : WebUILoginView(WebViewSettings()),
+    : WebUILoginView(BuildSettings()),
       screen_locker_(screen_locker),
       network_state_helper_(new login::NetworkStateHelper),
       weak_factory_(this) {
   set_should_emit_login_prompt_visible(false);
   ash::WmShell::Get()->AddLockStateObserver(this);
   ash::WmShell::Get()->AddShellObserver(this);
   display::Screen::GetScreen()->AddObserver(this);
   DBusThreadManager::Get()->GetPowerManagerClient()->AddObserver(this);
 
   if (keyboard::KeyboardController::GetInstance()) {
@@ skipping 318 matching lines @@
   }
 
   if (GetOobeUI()) {
     const gfx::Size& size = primary_display.size();
     GetOobeUI()->GetCoreOobeActor()->SetClientAreaSize(size.width(),
                                                        size.height());
   }
 }
 
 }  // namespace chromeos