[ios clean] Remove stop on Browser coordinator dealloc

ios/clean/chrome/browser/ui/tools/tools_coordinator.mm

Index: ios/clean/chrome/browser/ui/tools/tools_coordinator.mm
diff --git a/ios/clean/chrome/browser/ui/tools/tools_coordinator.mm b/ios/clean/chrome/browser/ui/tools/tools_coordinator.mm
index d80614efade9ba785b91fb331f4e7ef8259e022d..489008b9e33a3c092abfbe8af3ec71d06932506e 100644
--- a/ios/clean/chrome/browser/ui/tools/tools_coordinator.mm
+++ b/ios/clean/chrome/browser/ui/tools/tools_coordinator.mm
@@ -38,9 +38,8 @@
 }
 
 - (void)stop {
-  [self.menuViewController.presentingViewController

[edchin, 2017/01/31 07:18:26]

At cursory glance, this seems right to me since it has symmetry in the start and
stop methods. I wonder if there is something else that is causing the corrupted
state?

-      dismissViewControllerAnimated:self.context.animated
-                         completion:nil];
+  [self.menuViewController dismissViewControllerAnimated:self.context.animated
+                                              completion:nil];

[edchin, 2017/01/31 07:18:26]

Can you explain how this fixes the corrupted state?

 }
 
 #pragma mark - UIViewControllerTransitioningDelegate