[GRC] Coordination Unit Graph Observer

services/resource_coordinator/coordination_unit/coordination_unit_impl_unittest_util.cc

Index: services/resource_coordinator/coordination_unit/coordination_unit_impl_unittest_util.cc
diff --git a/services/resource_coordinator/coordination_unit/coordination_unit_impl_unittest_util.cc b/services/resource_coordinator/coordination_unit/coordination_unit_impl_unittest_util.cc
index c210b963525b2e80d97d81d65c5dc506073f71b3..8568649c521a594f9971b4b0ebedcf0d2ded8181 100644
--- a/services/resource_coordinator/coordination_unit/coordination_unit_impl_unittest_util.cc
+++ b/services/resource_coordinator/coordination_unit/coordination_unit_impl_unittest_util.cc
@@ -20,7 +20,11 @@ void OnLastServiceRefDestroyed() {
 
 CoordinationUnitImplTestBase::CoordinationUnitImplTestBase()
     : service_ref_factory_(base::Bind(&OnLastServiceRefDestroyed)),
-      provider_(&service_ref_factory_) {}
+      provider_(&service_ref_factory_, &coordination_unit_manager_) {
+  // Ensure tests only include the observers that are needed instead
+  // of what is registered by default to run in a browser instance.
+  coordination_unit_manager_.UnregisterAllObserversForTesting();

[Zhen Wang, 2017/06/15 20:07:53]

Is there another test helper class that can include the default ones? I am not
sure if we need those for integration test or something similar.

[matthalp, 2017/06/16 00:01:27]

I have gone ahead and removed this function. Instead,
CoordinationUnitManager::Register can be called within the
ResourceCoordinatorService constructor, which owns CoordinationUnitManager
throughout it's lifetime (i.e. the ResourceCoordinatorService's lifetime).

The comment about registering an observer in CoordinationUnitGraphObserver has
been updated.

+}
 
 CoordinationUnitImplTestBase::~CoordinationUnitImplTestBase() = default;