Use MacViews for ContentSettingBubbles.

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

Index: chrome/browser/ui/cocoa/location_bar/content_setting_bubble_views_mac.h
diff --git a/chrome/browser/ui/cocoa/location_bar/content_setting_bubble_views_mac.h b/chrome/browser/ui/cocoa/location_bar/content_setting_bubble_views_mac.h
new file mode 100644
index 0000000000000000000000000000000000000000..0b24e9cc27f309ae6f75b915e5a85bb1336002f2
--- /dev/null
+++ b/chrome/browser/ui/cocoa/location_bar/content_setting_bubble_views_mac.h
@@ -0,0 +1,25 @@
+// Copyright (c) 2016 The Chromium Authors. All rights reserved.

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

nit: no `(c)`, more in other files.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+

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

#include guards?

also stuff under c/b/ui/cocoa doesn't usually need a _mac suffix

+#include "ui/gfx/geometry/point.h"
+#include "ui/views/widget/widget.h"
+
+class ContentSettingBubbleModel;
+namespace content {
+class WebContents;
+}
+
+// This class bridges a views ContentSettingBubbleContents so it can be used by
+// the Cocoa location bar code.
+class ContentSettingBubbleBridge {
+ public:
+  ContentSettingBubbleBridge(ContentSettingBubbleModel* model,

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

I think this can just be a standalone function that also does the call to
Show()? (it would be different if the destructor had a task that needed doing,
but I think it's effectively a no-op here).

perhaps even in browser_dialogs.h / browser_dialogs_views_mac.cc 

Edit: that might disrupt a later comment wrt the SetAnchorRect call. So, new
idea: make this static function [include the Show() call] on a class like
`ContentSettingBubbleBridgeMac`. Forward-declare ContentSettingBubbleBridgeMac
in content_setting_bubble_contents.h and make it a friend of
ContentSettingBubbleContents.

Note: I still think this "class-with-single-static-method" should still be
declared in browser_dialogs.h / browser_dialogs_views_mac.cc . It _could_ stay
here, but it would be a little bit naughty: forward-declaring a class
implemented under c/b/ui/cocoa to be a friend of a class in c/b/ui/views
technically breaks DEPS. I a friend class in browse_dialogs.h is a nice
alternative to run by the c/b/ui/views OWNERS [versus modifying/overloading the
ContentSettingBubbleContents constructor to accept a gfx::Point instead of a
views::View].

+                             content::WebContents* web_contents,
+                             const gfx::Point& anchor);
+  ~ContentSettingBubbleBridge();
+
+  void Show();
+ private:

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

nit: blank line before

+  views::Widget* bubble_widget_;
+};

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

nit: DISALLOW_COPY_AND_ASSIGN(..)