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

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

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 23 matching lines @@
 
 // The views implementation of the website settings UI.
 class WebsiteSettingsPopupView
     : public PermissionSelectorViewObserver,
       public views::BubbleDelegateView,
       public views::ButtonListener,
       public views::LinkListener,
       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();
 
  private:
   WebsiteSettingsPopupView(views::View* anchor_view,
                            Profile* profile,
                            content::WebContents* web_contents,
                            const GURL& url,
                            const content::SSLStatus& ssl);
 
@@ skipping 109 matching lines @@
   views::Link* site_settings_link_;
 
   views::View* connection_info_content_;
 
   base::WeakPtrFactory<WebsiteSettingsPopupView> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebsiteSettingsPopupView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_WEBSITE_SETTINGS_POPUP_VIEW_H_