Create Login UI for Demo User.

chrome/browser/chromeos/kiosk_mode/kiosk_mode_screensaver.cc

 // 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.
 
 #include "chrome/browser/chromeos/kiosk_mode/kiosk_mode_screensaver.h"
 
+#include "base/bind.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "chrome/browser/chromeos/dbus/dbus_thread_manager.h"
+#include "chrome/browser/chromeos/kiosk_mode/kiosk_mode_helper.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/ui/screensaver_extension_dialog.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "content/public/browser/notification_service.h"
 
-namespace {
-
-// This (along with screensaver location) will change once we have the
-// retail mode enterprise policy set up.
-const int64 kScreensaverIdleTimeout = 60;
-
-}  // namespace
-
 namespace chromeos {
 
 KioskModeScreensaver::KioskModeScreensaver() {
+  if (chromeos::KioskModeHelper::Get()->is_initialized())
+    Init();

[xiyuan, 2012/02/29 18:01:12]

nit: Could we have a better name? On first look, "Init" confuses me with the
Initialize method on KioskModHelper. How about change it to something else, e.g.
"Setup".

[rkc, 2012/02/29 20:57:48]

Done.

+  else
+    chromeos::KioskModeHelper::Get()->Initialize(

[xiyuan, 2012/02/29 18:01:12]

nit: suggest to enclose this with {} since it's more than one line.

[rkc, 2012/02/29 20:57:48]

Done.

+        base::Bind(&KioskModeScreensaver::Init,
+                   base::Unretained(this)));
+
+}
+
+KioskModeScreensaver::~KioskModeScreensaver() {
+  chromeos::PowerManagerClient* power_manager =
+      chromeos::DBusThreadManager::Get()->GetPowerManagerClient();
+  if (power_manager->HasObserver(this))
+    power_manager->RemoveObserver(this);
+}
+
+void KioskModeScreensaver::Init() {
+
   // We should NOT be created if already logged in.
   CHECK(!chromeos::UserManager::Get()->user_is_logged_in());
 
   registrar_.Add(this, chrome::NOTIFICATION_LOGIN_USER_CHANGED,
                  content::NotificationService::AllSources());
 
   // We will register ourselves now and unregister if a user logs in.
   chromeos::PowerManagerClient* power_manager =
       chromeos::DBusThreadManager::Get()->GetPowerManagerClient();
   if (!power_manager->HasObserver(this))
     power_manager->AddObserver(this);
 
   // Register for the next Idle for kScreensaverIdleTimeout event.
-  chromeos::DBusThreadManager::Get()->GetPowerManagerClient()->
-      RequestIdleNotification(kScreensaverIdleTimeout * 1000);
-}
-
-KioskModeScreensaver::~KioskModeScreensaver() {
-  chromeos::PowerManagerClient* power_manager =
-      chromeos::DBusThreadManager::Get()->GetPowerManagerClient();
-  if (power_manager->HasObserver(this))
-    power_manager->RemoveObserver(this);
+  chromeos::DBusThreadManager::Get()->

[Nikita (slow), 2012/02/29 12:21:55]

I've realized that you would also need a patch for PowerManager. If Chromebook
stays idle at the login screen it will power off after some time. In Kiosk mode
we want to disable such behavior.

[rkc, 2012/02/29 20:57:48]

That is correct. I need to speak to Benson about this. The disable_idle_suspend
option still dims the screen, which is also something we want to disable for
this.

I'll address this in a forthcoming CL and changes to power manager.

+      GetPowerManagerClient()->RequestIdleNotification(
+          chromeos::KioskModeHelper::Get()->GetScreensaverTimeout() * 1000);
 }
 
 // NotificationObserver overrides:
 void KioskModeScreensaver::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   DCHECK_EQ(type, chrome::NOTIFICATION_LOGIN_USER_CHANGED);
   // User logged in, remove our observers, screensaver will be deactivated.
   chromeos::PowerManagerClient* power_manager =
       chromeos::DBusThreadManager::Get()->GetPowerManagerClient();
   if (power_manager->HasObserver(this))
     power_manager->RemoveObserver(this);
 
   ShutdownKioskModeScreensaver();
 }
 
 void KioskModeScreensaver::IdleNotify(int64 threshold) {
   // We're idle, next time we go active, we need to know so we can remove
   // the logout dialog if it's still up.

[Nikita (slow), 2012/02/29 12:21:55]

nit: logout dialog?

[rkc, 2012/02/29 20:57:48]

Done.

-  chromeos::DBusThreadManager::Get()->GetPowerManagerClient()->
-      RequestActiveNotification();
+  chromeos::DBusThreadManager::Get()->
+      GetPowerManagerClient()->RequestActiveNotification();
 
   browser::ShowScreensaverDialog();
 }
 
 void KioskModeScreensaver::ActiveNotify() {
   // Before anything else, close the logout dialog to prevent restart

[Nikita (slow), 2012/02/29 12:21:55]

nit: logout dialog?

[rkc, 2012/02/29 20:57:48]

Done.

   browser::CloseScreensaverDialog();
 
   // Now that we're active, register a request for notification for
   // the next time we go idle for kScreensaverIdleTimeout seconds.
-  chromeos::DBusThreadManager::Get()->GetPowerManagerClient()->
-      RequestIdleNotification(kScreensaverIdleTimeout * 1000);
-}
+  chromeos::DBusThreadManager::Get()->
+      GetPowerManagerClient()->RequestIdleNotification(
+          chromeos::KioskModeHelper::Get()->GetScreensaverTimeout() * 1000);}
 
 static KioskModeScreensaver* g_kiosk_mode_screensaver = NULL;
 
 void InitializeKioskModeScreensaver() {
   if (g_kiosk_mode_screensaver) {
     LOG(WARNING) << "Screensaver was already initialized";
     return;
   }
 
   g_kiosk_mode_screensaver = new KioskModeScreensaver();
 }
 
 void ShutdownKioskModeScreensaver() {
-  if (g_kiosk_mode_screensaver) {
+  if (!g_kiosk_mode_screensaver) {

[Nikita (slow), 2012/02/29 12:21:55]

On the second thought delete of a NULL pointer is a noop so you may as well
remove this is clause and remove check on shutdown path to simplify.

[rkc, 2012/02/29 20:57:48]

This actually really helped me with debugging already :) I'm inclined to leave
it in just to catch any weird places where Shutdown gets called without
Initialize, which should never really happen.

     LOG(WARNING) << "Screensaver shutdown called when uninitialized.";
     return;
   }
 
   delete g_kiosk_mode_screensaver;
   g_kiosk_mode_screensaver = NULL;
 }
 
 }  // namespace chromeos