session_manager: Create ChromeSessionManager early

chrome/browser/chromeos/login/session/kiosk_auto_launcher_session_initializer.h

+// Copyright 2014 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_SESSION_KIOSK_AUTO_LAUNCHER_SESSION_INITIALIZER_H_
+#define CHROME_BROWSER_CHROMEOS_LOGIN_SESSION_KIOSK_AUTO_LAUNCHER_SESSION_INITIALIZER_H_
+
+#include "base/macros.h"
+
+namespace chromeos {
+
+class KioskAutoLauncherSessionInitializer {
+ public:
+  KioskAutoLauncherSessionInitializer();
+  ~KioskAutoLauncherSessionInitializer();
+
+  void Start();

[achuithb, 2016/10/27 19:12:45]

This is really just a static C-style function invocation, right? The class
doesn't really do anything.

Should we be creating separate classes for each of these Start() methods?

[achuithb, 2016/10/27 19:13:32]

Why not StartKioskSession(), StartOOBESession(),
StartRestoreAfterCrashSession(), StartStubLoginSession()

[xiyuan, 2016/10/27 21:10:43]

Agree.

Removed those initializer classes and moved the code into functions into
chrome_session_manager.cc.

+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(KioskAutoLauncherSessionInitializer);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_LOGIN_SESSION_KIOSK_AUTO_LAUNCHER_SESSION_INITIALIZER_H_