Add a basic demo mode browser test.

chrome/browser/chromeos/login/parallel_authenticator.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_PARALLEL_AUTHENTICATOR_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_PARALLEL_AUTHENTICATOR_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 114 matching lines @@
   // Initiates incognito ("browse without signing in") login.
   // Mounts tmpfs and notifies consumer on the success/failure.
   virtual void LoginOffTheRecord() OVERRIDE;
 
   // Initiates login into the public account identified by |username|.
   // Mounts an ephemeral cryptohome and notifies consumer on the
   // success/failure.
   virtual void LoginAsPublicAccount(const std::string& username) OVERRIDE;
 
   // Initiates login into the kiosk mode account identified by |app_user_id|.
-  // Mounts an public but non-ephemeral cryptohome and notifies consumer on the
-  // success/failure.
-  virtual void LoginAsKioskAccount(
-      const std::string& app_user_id, bool force_ephemeral) OVERRIDE;
+  // Mounts an public and notifies consumer on the success/failure. If

[bartfab (slow), 2014/03/20 13:22:29]

"Mounts an public" is missing a few words :).

[rkc, 2014/03/20 21:45:01]

Done.

+  // |force_guest| is true, instead we mount a guest mount (which is

[bartfab (slow), 2014/03/20 13:22:29]

"If [...], instead we [...], otherwise the mount [...]." is both grammatically
wrong (change of subject mid-sentence) and very hard to follow. Why not say
something like:

If |force_ephemeral| is true, the mount is always ephemeral. Otherwise, the
mount is ephemeral iff the |DeviceEphemeralUsersEnabled| policy is enabled.

[rkc, 2014/03/20 21:45:01]

Changed the comment around to make it clearer.

+  // ephemeral), otherwise the mount is non-ephemeral (unless specified
+  // by enterprise policy that it should be ephemeral).
+  virtual void LoginAsKioskAccount(const std::string& app_user_id,
+                                   bool force_guest) OVERRIDE;
 
   // These methods must be called on the UI thread, as they make DBus calls
   // and also call back to the login UI.
   virtual void OnRetailModeLoginSuccess() OVERRIDE;
   virtual void OnLoginSuccess() OVERRIDE;
   virtual void OnLoginFailure(const LoginFailure& error) OVERRIDE;
   virtual void RecoverEncryptedData(
       const std::string& old_password) OVERRIDE;
   virtual void ResyncEncryptedData() OVERRIDE;
 
@@ skipping 95 matching lines @@
   // When |remove_user_data_on_failure_| is set, we delay calling
   // consumer_->OnLoginFailure() until we removed the user cryptohome.
   const LoginFailure* delayed_login_failure_;
 
   DISALLOW_COPY_AND_ASSIGN(ParallelAuthenticator);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_PARALLEL_AUTHENTICATOR_H_