[GRC] Coordination Unit Graph Observer

services/resource_coordinator/coordination_unit/coordination_unit_manager.h

 // Copyright 2017 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 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 {
 class BinderRegistry;
 class ServiceContextRefFactory;
 }  // service_manager
 
 namespace resource_coordinator {
 
 class CoordinationUnitImpl;
 class CoordinationUnitGraphObserver;
 
 // The CoordinationUnitManager is a singleton that encapsulates all
-// aspects of Coordination Units within the ResourceCoordinatorService.
+// aspects of Coordination Units within the resource_coordinator service.
 // All functionality for dealing with CoordinationUnits should be contained
 // within this class or classes that are owned by it
 class CoordinationUnitManager {
  public:
   CoordinationUnitManager();
   ~CoordinationUnitManager();
 
   void OnStart(service_manager::BinderRegistry* registry,
                service_manager::ServiceContextRefFactory* service_ref_factory);
   void RegisterObserver(
-      std::unique_ptr<CoordinationUnitGraphObserver> observers);
+      std::unique_ptr<CoordinationUnitGraphObserver> observer);
+  void OnCoordinationUnitCreated(CoordinationUnitImpl* coordination_unit);
+  void OnCoordinationUnitWillBeDestroyed(
+      CoordinationUnitImpl* coordination_unit);
 
-  void NotifyObserversCoordinationUnitCreated(
-      CoordinationUnitImpl* coordination_unit);
-  void NotifyObserversCoordinationUnitWillBeDestroyed(
-      CoordinationUnitImpl* coordination_unit);
+  std::vector<std::unique_ptr<CoordinationUnitGraphObserver>>& observers() {

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

nit: looks this is just for testing? maybe const std::vector<blah>&
observers_for_testing()

+    return observers_;
+  }
 
  private:
   std::vector<std::unique_ptr<CoordinationUnitGraphObserver>> observers_;
 
   static void Create(
       service_manager::ServiceContextRefFactory* service_ref_factory);
 
   DISALLOW_COPY_AND_ASSIGN(CoordinationUnitManager);
 };
 
 }  // namespace resource_coordinator
 
 #endif  // SERVICES_RESOURCE_COORDINATOR_COORDINATION_UNIT_COORDINATION_UNIT_MANAGER_H_