Dismiss popups on Tab deselected

ios/chrome/browser/ui/browser_view_controller.mm

 // Copyright 2012 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.
 
 #import "ios/chrome/browser/ui/browser_view_controller.h"
 
 #import <AssetsLibrary/AssetsLibrary.h>
 #import <MobileCoreServices/MobileCoreServices.h>
 #import <PassKit/PassKit.h>
 #import <Photos/Photos.h>
@@ skipping 3822 matching lines @@
 }
 
 #pragma mark - Command Handling
 
 - (IBAction)chromeExecuteCommand:(id)sender {
   NSInteger command = [sender tag];
 
   if (!_model || !_browserState)
     return;
 
+  [self dismissPopups];

[marq (ping after 24h), 2017/03/02 06:07:40]

I can't verify that this is correct.

What are all the paths by which the BVC's chromeExecuteCommand can be called? Is
it correct in every one of those cases to dismiss popups? I don't know. 

Remember that this code path can be triggered in many different ways.

+
   switch (command) {
     case IDC_BACK:
       [[_model currentTab] goBack];
       break;
     case IDC_BOOKMARK_PAGE:
       [self initializeBookmarkInteractionController];
       [_bookmarkInteractionController
           presentBookmarkForTab:[_model currentTab]
             currentlyBookmarked:_toolbarModelIOS->IsCurrentTabBookmarkedByUser()
                          inView:[_toolbarController bookmarkButtonView]
@@ skipping 1172 matching lines @@
 
 - (UIView*)voiceSearchButton {
   return _voiceSearchButton;
 }
 
 - (id<LogoAnimationControllerOwner>)logoAnimationControllerOwner {
   return [self currentLogoAnimationControllerOwner];
 }
 
 @end