Use MacViews for ContentSettingBubbles.

chrome/browser/ui/cocoa/location_bar/content_setting_bubble_views_mac.mm

+// Copyright (c) 2016 The Chromium Authors. All rights reserved.

[tapted, 2016/01/11 02:50:57]

can this be .cc instead of .mm?

+// 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/location_bar/content_setting_bubble_views_mac.h"

[tapted, 2016/01/11 02:50:57]

nit: blank line after the #include for a .cc's corresponding .h file

+#include "chrome/browser/ui/views/content_setting_bubble_contents.h"
+
+ContentSettingBubbleBridge::ContentSettingBubbleBridge(
+    ContentSettingBubbleModel* model,
+    content::WebContents* web_contents,
+    const gfx::Point& anchor) {
+  ContentSettingBubbleContents* contents =
+      new ContentSettingBubbleContents(model,
+          web_contents, NULL,

[tapted, 2016/01/11 02:50:57]

nit: NULL -> nullptr

+          views::BubbleBorder::Arrow::TOP_RIGHT);

[tapted, 2016/01/11 02:50:57]

I *think* CreateBubbleWidget(..) in ui/views/bubble/bubble_delegate.cc should be
setting a value from the parent_window() when initializing the Widget. This sets
up a kind of ownership (e.g. when the parent window closes), ensures the bubble
is always layered above the parent, and (ignoring resizes) ensures it is "stuck"
to the anchor when the window moves around. Without it, I think there will be
weird behaviour, e.g., when the window is fading out or when clicking the
location bar decoration multiple times.

It looks like we might be able to just call
contents->set_parent_window(web_contents->GetTopLevelNativeWindow()) after this

+  contents->SetExplicitAnchorRect(gfx::Rect(anchor, gfx::Size(0, 0)));

[tapted, 2016/01/11 02:50:57]

nit: Size(0, 0) -> Size()

+  bubble_widget_ = views::BubbleDelegateView::CreateBubble(contents);
+}
+
+ContentSettingBubbleBridge::~ContentSettingBubbleBridge() {}
+
+void ContentSettingBubbleBridge::Show() {
+  bubble_widget_->Show();
+}