Implement ephemeral users

chrome/browser/chromeos/login/wizard_controller.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/login/wizard_controller.h"
 
 #include <signal.h>
 #include <sys/types.h>
 
 #include <string>
@@ skipping 245 matching lines @@
   oobe_display_ = NULL;
 }
 
 void WizardController::ShowUpdateScreen() {
   VLOG(1) << "Showing update screen.";
   SetStatusAreaVisible(true);
   SetCurrentScreen(GetUpdateScreen());
 }
 
 void WizardController::ShowUserImageScreen() {
+  // Skip image selection for ephemeral users.
+  if (chromeos::UserManager::Get()->current_user_is_ephemeral()) {

[Nikita (slow), 2012/03/07 10:03:04]

I think we should alter UX for these users as less as possible. They are the
same as normal users but all their data is removed on sign out.

Not sure with this one. Was this decided?

[use bartfab instead, 2012/03/07 11:10:08]

I made the change as requested but I think skipping image selection is the
better UX:

Setting an image is a once-off necessity for regular users. Should ephemeral
users not be presented with the most common UX path, i.e. that of a returning
regular Chrome OS user?

+    OnUserImageSkipped();
+    return;
+  }
   VLOG(1) << "Showing user image screen.";
   SetStatusAreaVisible(false);
   SetCurrentScreen(GetUserImageScreen());
   host_->SetShutdownButtonEnabled(false);
 }
 
 void WizardController::ShowEulaScreen() {
   VLOG(1) << "Showing EULA screen.";
   SetStatusAreaVisible(false);
   SetCurrentScreen(GetEulaScreen());
@@ skipping 432 matching lines @@
 
 bool WizardController::usage_statistics_reporting() const {
   return usage_statistics_reporting_;
 }
 
 void WizardController::SetZeroDelays() {
   kShowDelayMs = 0;
 }
 
 }  // namespace chromeos