[GRC] Coordination Unit Graph Observer

services/resource_coordinator/coordination_unit/coordination_unit_manager.h

Index: services/resource_coordinator/coordination_unit/coordination_unit_manager.h
diff --git a/services/resource_coordinator/coordination_unit/coordination_unit_manager.h b/services/resource_coordinator/coordination_unit/coordination_unit_manager.h
index 7d07f8bbdf5f2140bde0018f9d7a3bb46daa57d4..0907d05e2a52a32fe9176d8586f6719268d356ed 100644
--- a/services/resource_coordinator/coordination_unit/coordination_unit_manager.h
+++ b/services/resource_coordinator/coordination_unit/coordination_unit_manager.h
@@ -5,6 +5,9 @@
 #ifndef SERVICES_RESOURCE_COORDINATOR_COORDINATION_UNIT_COORDINATION_UNIT_MANAGER_H_
 #define SERVICES_RESOURCE_COORDINATOR_COORDINATION_UNIT_COORDINATION_UNIT_MANAGER_H_
 
+#include <memory>
+#include <vector>
+
 #include "base/macros.h"
 
 namespace service_manager {
@@ -14,6 +17,9 @@ class ServiceContextRefFactory;
 
 namespace resource_coordinator {
 
+class CoordinationUnitImpl;
+class CoordinationUnitGraphObserver;
+
 // The CoordinationUnitManager is a singleton that encapsulates all
 // aspects of Coordination Units within the ResourceCoordinatorService.

[oystein (OOO til 10th of July), 2017/06/20 18:39:24]

nit: resource_coordinator service.

[matthalp, 2017/06/20 19:02:52]

Done.

 // All functionality for dealing with CoordinationUnits should be contained
@@ -25,8 +31,17 @@ class CoordinationUnitManager {
 
   void OnStart(service_manager::BinderRegistry* registry,
                service_manager::ServiceContextRefFactory* service_ref_factory);
+  void RegisterObserver(
+      std::unique_ptr<CoordinationUnitGraphObserver> observers);
+
+  void NotifyObserversCoordinationUnitCreated(

[lpy, 2017/06/19 23:21:39]

OnCoordinationUnitCreated

[matthalp, 2017/06/20 19:02:52]

Done.

+      CoordinationUnitImpl* coordination_unit);
+  void NotifyObserversCoordinationUnitWillBeDestroyed(

[lpy, 2017/06/19 23:21:39]

OnCoordinationUnitWillBeDestroyed

[matthalp, 2017/06/20 19:02:53]

Done. Agree that this is more consistent naming with OnCoordinationUnitCreated.

+      CoordinationUnitImpl* coordination_unit);
 
  private:
+  std::vector<std::unique_ptr<CoordinationUnitGraphObserver>> observers_;
+
   static void Create(
       service_manager::ServiceContextRefFactory* service_ref_factory);