session_manager: Add SessionManagerObserver interface

components/session_manager/core/session_manager_observer.h

+// Copyright 2016 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 COMPONENTS_SESSION_MANAGER_CORE_SESSION_MANAGER_OBSERVER_H_
+#define COMPONENTS_SESSION_MANAGER_CORE_SESSION_MANAGER_OBSERVER_H_
+
+#include "components/session_manager/session_manager_types.h"
+
+namespace session_manager {
+
+// An observer interface for SessionManager.
+// TODO(xiyuan): Use this to replace UserManager::UserSessionStateObserver.

[achuithb, 2016/11/17 00:31:23]

Is there a bug that could be referenced here?

[xiyuan, 2016/11/17 16:57:24]

Done. Used my refactoring tracking bug http://crbug.com/657149 here.

+class SessionManagerObserver {
+ public:
+  // Invoked when session state is changed.
+  virtual void OnSessionStateChanged() = 0;
+};

[achuithb, 2016/11/17 00:31:23]

Let's add
public:
  SessionManagerObserver() = default;
protected:
  virtual ~SessionManagerObserver() = default;
private:
  DISALLOW_COPY_AND_ASSIGN(SessionManagerObserver);

[xiyuan, 2016/11/17 16:57:24]

Added the virtual dtor and DISALLOW_COPY_AND_ASSIGN.

Is creating a constructor for interface class recommended now?

+
+}  // namespace session_manager
+
+#endif  // COMPONENTS_SESSION_MANAGER_CORE_SESSION_MANAGER_OBSERVER_H_