Implement permission bubble view for Cocoa.

chrome/browser/ui/cocoa/website_settings/permission_bubble_cocoa.mm

Index: chrome/browser/ui/cocoa/website_settings/permission_bubble_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/website_settings/permission_bubble_cocoa.mm b/chrome/browser/ui/cocoa/website_settings/permission_bubble_cocoa.mm
new file mode 100644
index 0000000000000000000000000000000000000000..c90c8dbf9bded4dab6b5273b5bb0dd925bc85fa9
--- /dev/null
+++ b/chrome/browser/ui/cocoa/website_settings/permission_bubble_cocoa.mm
@@ -0,0 +1,59 @@
+// Copyright 2014 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.
+
+#include "chrome/browser/ui/cocoa/website_settings/permission_bubble_cocoa.h"
+
+#import "chrome/browser/ui/cocoa/base_bubble_controller.h"
+#import "chrome/browser/ui/cocoa/browser_window_controller.h"
+#import "chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.h"
+#import "chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.h"
+#import "chrome/browser/ui/website_settings/permission_bubble_view.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/browser/web_contents_view.h"
+
+PermissionBubbleCocoa::PermissionBubbleCocoa(NSWindow* parent_window)
+    : parent_window_(parent_window), delegate_(NULL), bubbleController_(nil) {}
+
+PermissionBubbleCocoa::~PermissionBubbleCocoa() {
+  if (delegate_)
+    delegate_->SetView(NULL);
+}
+
+void PermissionBubbleCocoa::Show(
+    const std::vector<PermissionBubbleRequest*>& requests,
+    const std::vector<bool>& accept_state,
+    bool customization_mode) {
+  DCHECK(parent_window_);
+
+  if (!bubbleController_) {
+    bubbleController_ = [[PermissionBubbleController alloc]
+        initWithParentWindow:parent_window_
+                      bridge:this];
+  }
+
+  LocationBarViewMac* location_bar =
+      [[parent_window_ windowController] locationBarBridge];
+  NSPoint anchor = location_bar->GetPageInfoBubblePoint();
+  [bubbleController_ showAtAnchor:[parent_window_ convertBaseToScreen:anchor]
+                     withDelegate:delegate_
+                      forRequests:requests
+                     acceptStates:accept_state
+                customizationMode:customization_mode];
+}
+
+void PermissionBubbleCocoa::Hide() {
+  [bubbleController_ close];
+}
+
+void PermissionBubbleCocoa::SetDelegate(Delegate* delegate) {
+  if (delegate_ == delegate)

[groby-ooo-7-16, 2014/03/03 18:34:05]

I'm confused - this seems to re-attach an existing bubble to a new delegate?

Doesn't that leave contents from the previous bubble around? I'd feel better if
we closed and re-opened a bubble when a delegate changes.

[leng, 2014/03/03 19:01:04]

It's just here as an early return.  The delegate itself should control the
closing and re-showing of the bubbles.

+    return;
+  if (delegate_ && delegate)
+    delegate_->SetView(NULL);
+  delegate_ = delegate;
+}
+
+void PermissionBubbleCocoa::OnBubbleClosing() {
+  bubbleController_ = nil;

[groby-ooo-7-16, 2014/03/03 18:34:05]

delegate_->SetView(NULL) here? I'm not clear on the lifetimes of the involved
objects. Maybe document in header?

[leng, 2014/03/03 19:01:04]

No, because the delegate might still have 'control' of the view, and re-open it
with a different request.

+}