[Mac] Address buggy permission bubble behaviour on dismissal via ESC.

chrome/browser/ui/cocoa/browser_window_controller.mm

Index: chrome/browser/ui/cocoa/browser_window_controller.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller.mm b/chrome/browser/ui/cocoa/browser_window_controller.mm
index 3f9f5a88871ab750d2c746cd009394b49cfe2330..ff317cb72a7d979d98d267f2e01b10ce87ea3d0b 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller.mm
@@ -1593,7 +1593,7 @@ bool IsTabDetachingInFullscreenEnabled() {
 - (void)dismissPermissionBubble {
   PermissionRequestManager* manager = [self permissionRequestManager];
   if (manager)
-    manager->HideBubble();
+    manager->DismissBubble();

[raymes, 2016/10/27 01:47:52]

Actually, can we just cast manager to a PermissionPrompt::Delegate here and call
closing? That would avoid needing to add DismissBubble at all. IsBubbleVisible
is also public so can be called if it's needed but see my other question.

[dominickn, 2016/10/27 02:09:02]

I asked felt this question. Since you're okay with it as a permissions OWNER,
I'm going to implement it.

 }
 
 // Nil out the weak translate bubble controller reference.