Remove code in subclasses already performed by the base class.

ios/clean/chrome/browser/ui/tab_grid/tab_grid_coordinator.mm

Index: ios/clean/chrome/browser/ui/tab_grid/tab_grid_coordinator.mm
diff --git a/ios/clean/chrome/browser/ui/tab_grid/tab_grid_coordinator.mm b/ios/clean/chrome/browser/ui/tab_grid/tab_grid_coordinator.mm
index 1962c7aa015a3bdd92cd0cb4bccce53905da5717..d6bfc4719d75412fd4145d277dbab65a7b9fa63d 100644
--- a/ios/clean/chrome/browser/ui/tab_grid/tab_grid_coordinator.mm
+++ b/ios/clean/chrome/browser/ui/tab_grid/tab_grid_coordinator.mm
@@ -86,8 +86,9 @@
   [super stop];
   [self.browser->dispatcher() stopDispatchingToTarget:self];
   [self.mediator disconnect];
+  // PLACEHOLDER: Remove child coordinators here for now. This might be handled
+  // differently later on.
   for (BrowserCoordinator* child in self.children) {

[marq (ping after 24h), 2017/04/18 12:13:26]

Update this for coordinator changes?

[lpromero, 2017/05/03 13:13:23]

Was this comment the same as the previous one? Otherwise what did you mean by
"coordinator changes"?

[marq (ping after 24h), 2017/05/03 13:17:49]

I thought we were handling this (removing children) in the base class's -stop
method. I was probably confused.

[lpromero, 2017/05/03 13:20:54]

We stop childrens in the parent's stop, but we only remove children from the
parent's dealloc.

-    [child stop];
     [self removeChildCoordinator:child];
   }
 }