Add the ability to show a demo app on OOBE if a machine is derelict.

chrome/browser/chromeos/login/authenticator.h

Index: chrome/browser/chromeos/login/authenticator.h
diff --git a/chrome/browser/chromeos/login/authenticator.h b/chrome/browser/chromeos/login/authenticator.h
index f061803683285627b030c32763bdd50608946d6a..079fb61e09ce76ee2b36b2a9650cbdce14000564 100644
--- a/chrome/browser/chromeos/login/authenticator.h
+++ b/chrome/browser/chromeos/login/authenticator.h
@@ -58,7 +58,9 @@ class Authenticator : public base::RefCountedThreadSafe<Authenticator> {
 
   // Initiates login into kiosk mode account identified by |app_user_id|.
   // The |app_user_id| is a generated username for the account.
-  virtual void LoginAsKioskAccount(const std::string& app_user_id) = 0;
+  // |force_ephemeral| specifies whether to force the session to be ephemeral.

[bartfab (slow), 2014/02/13 19:51:53]

Here and elsewhere: Why is the term "force" part of this argument's name? It is
not like it forces a behavior, overriding some default. It switches between
ephemeral and non-ephemeral. It should simply be called |ephemeral|.

[rkc, 2014/02/13 23:22:01]

Not really. If this flag is not specified, the ephemeral behavior will be
determined by enterprise policy. This flag specifically 'forces' us to use an
ephemeral mount, regardless of the enterprise policy. Hence, the name is
appropriate.

+  virtual void LoginAsKioskAccount(
+      const std::string& app_user_id, bool force_ephemeral) = 0;

[bartfab (slow), 2014/02/13 19:51:53]

This violates the style guide. Either the entire declaration fits on one line or
each argument goes on a separate line.

[rkc, 2014/02/13 23:22:01]

Done.

 
   // Completes retail mode login.
   virtual void OnRetailModeLoginSuccess() = 0;