arc: Start ARC for Public Session users.

components/arc/arc_util.h

Index: components/arc/arc_util.h
diff --git a/components/arc/arc_util.h b/components/arc/arc_util.h
index 7558ac3944d517a869b96bfb36a49f7b1d2dee75..01b42a94cec5faee2a96b475e1a6659247385101 100644
--- a/components/arc/arc_util.h
+++ b/components/arc/arc_util.h
@@ -78,6 +78,16 @@ void SetArcAvailableCommandLineForTesting(base::CommandLine* command_line);
 // should also return true in that case.
 bool IsArcKioskMode();

[Luis Héctor Chávez, 2017/08/29 16:24:09]

This _should_ now be unused. Can you remove it?

There are also mentions of kiosk mode in
https://cs.chromium.org/chromium/src/ash/metrics/user_metrics_recorder.cc?q=I...
, should those also be updated?

[Sergey Poromov, 2017/08/29 16:37:30]

IsArcKioskMode() is used in
https://cs.chromium.org/chromium/src/chrome/browser/chromeos/arc/arc_session_...
that should be done for Public Sessions.

What about UserMetricsRecorder - this code is in another component, so probably
the check should be left there.

 
+// Returns true if current user is a public session.
+// In that case ARC should also be started if available, using the same
+// authentication flow as ARC Kiosk mode.
+bool IsPublicSessionMode();

[Luis Héctor Chávez, 2017/08/29 16:24:09]

IIUC this is unused. Can you remove it?

[Sergey Poromov, 2017/08/29 16:37:30]

Done.

+
+// Returns true if current user is a robot account user.
+// These are Public Session and ARC Kiosk users.
+// The check is basically IsArcKioskMode() | IsPublicSessionMode().
+bool IsRobotAccountMode();
+
 // Returns true if ARC is allowed for the given user. Note this should not be
 // used as a signal of whether ARC is allowed alone because it only considers
 // user meta data. e.g. a user could be allowed for ARC but if the user signs in