[ios clean] Dismisses ToolsMenu on Layout changes.

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

Index: ios/clean/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm
diff --git a/ios/clean/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm b/ios/clean/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm
index bdfcad661eced3e263d3fcc6dd2f08f31d3be217..cdfa2c170d99777e5fa442cf3451bd4d4f766c8f 100644
--- a/ios/clean/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm
+++ b/ios/clean/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm
@@ -85,6 +85,12 @@
       UIEdgeInsetsMake(CGRectGetMaxY(self.toolbar.frame), 0, 0, 0);
 }
 
+- (void)viewWillLayoutSubviews {
+  // We need to dismiss the ToolsMenu everytime the Toolbar position changes
+  // (e.g. Size changes, rotation changes, etc.)

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

This is probably fine, but note that the conditions you list are not the
conditions for what triggers a layout. If any change is made to the view
hierarchy -- resizing a subview, for example -- that may also cause this to
trigger. 

[sczs, 2017/05/10 02:25:46]

Yes, you're right! I was stating what are we trying to achieve, or why are we
doing this. My own comment was showing me that something wasn't right. I've now
updated this.

+  [self.dispatcher closeToolsMenu];
+}
+
 #pragma mark - SettingsActions
 
 - (void)showSettings:(id)sender {