[Mac] Enable MacViews site settings bubble behind --enable-mac-views-dialogs.

chrome/browser/ui/views/website_settings/website_settings_popup_view.h

Index: chrome/browser/ui/views/website_settings/website_settings_popup_view.h
diff --git a/chrome/browser/ui/views/website_settings/website_settings_popup_view.h b/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
index 712f183bf1caae5ba633093441b8a5d08a95539b..1b931fd2acbfe3ab06047e37f8cc32b5b6692d71 100644
--- a/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
+++ b/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
@@ -41,13 +41,19 @@ class WebsiteSettingsPopupView
       public views::TabbedPaneListener,
       public WebsiteSettingsUI {
  public:
+  // To make the bubble's arrow point directly at the location icon rather than
+  // at the Omnibox's edge, the bubble's anchor rect will be insetted by this

[tapted, 2015/08/11 03:20:38]

nit: insetted -> inset is right I think.

But I think your discovery is the more pertinent comment to make. e.g.

// Margin built-in to the icon asset in the anchor view. Used to ensure the
// bubble arrow points to the actual icon.

[jackhou1, 2015/08/11 03:54:02]

Done.

+  // amount of pixels.
+  static const int kLocationIconVerticalMargin;
+
   ~WebsiteSettingsPopupView() override;
 
-  static void ShowPopup(views::View* anchor_view,
-                        Profile* profile,
-                        content::WebContents* web_contents,
-                        const GURL& url,
-                        const content::SSLStatus& ssl);
+  static views::BubbleDelegateView* ShowPopup(
+      views::View* anchor_view,
+      Profile* profile,
+      content::WebContents* web_contents,
+      const GURL& url,
+      const content::SSLStatus& ssl);
 
   static bool IsPopupShowing();