[GRC] ResourceCoordinatorInterface support to remove children

services/resource_coordinator/public/interfaces/coordination_unit.mojom

 // 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.
 
 module resource_coordinator.mojom;
 
 import "events.mojom";
 import "mojo/common/values.mojom";
 
 // Any new type here needs to be mirrored between coordination_unit_types.h and
@@ skipping 39 matching lines @@
   // a specific unit, so we don't have to deal with possibly-not-yet-created
   // children in AddChild()
   GetID() => (CoordinationUnitID id);
 
   // Add a new binding to an existing CoordinationUnit.
   AddBinding(CoordinationUnit& request);
 
   // child_id must represent a CU which already exists service-side,
   // and can't be a direct ascendent or descendent of the current unit.
   AddChild(CoordinationUnitID child_id);
+  RemoveChild(CoordinationUnitID child_id);

[nasko, 2017/06/09 22:19:13]

Does the above comment apply to RemoveChild? Reading it, likely not. If it
doesn't, please leave an empty line to clarify that the comment is only for
AddChild. Ideally add a comment that is applicable to RemoveChild.

[matthalp, 2017/06/09 22:31:58]

It does not. Comment added.

   SetCoordinationPolicyCallback(CoordinationPolicyCallback callback);
 
   // Sets a property on the CoordinationUnitImpl's internal key-value store.
   SetProperty(Property property);
 };