MacViews: support Views permission bubble

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

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_PERMISSIONS_BUBBLE_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_PERMISSIONS_BUBBLE_VIEW_H_
 
+#include <memory>
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "chrome/browser/ui/website_settings/permission_bubble_view.h"
+#include "ui/gfx/geometry/point.h"
 #include "ui/views/bubble/bubble_border.h"
 
 namespace views {
 class View;
 }
 
 class Browser;
 class PermissionsBubbleDialogDelegateView;
 
 class PermissionBubbleViewViews : public PermissionBubbleView {
  public:
-  explicit PermissionBubbleViewViews(Browser* browser);
+  // A PermissionBubbleViewViews owns an AnchorDelegate, which is passed in when
+  // constructing the PermissionBubbleViewViews. The AnchorDelegate is
+  // responsible for determining the anchor view (or point, for platforms
+  // without an appropriate native view) and the immediate parent NativeView of
+  // this view.
+  class AnchorDelegate {
+   public:
+    AnchorDelegate(){}

[tapted, 2016/05/03 12:02:39]

nit: space before {}

[Elly Fong-Jones, 2016/05/10 21:21:02]

Done.

+    virtual ~AnchorDelegate(){}

[tapted, 2016/05/03 12:02:39]

I think we can omit the destructor altogether for pure interfaces

[Elly Fong-Jones, 2016/05/10 21:21:02]

Then the destructor isn't virtual and can't be overridden in the children, it
seems.

[tapted, 2016/05/11 07:06:18]

I don't think we need to override it in the children either :). (but if we ever
did, declaring it `virtual` rather than `override` in the children should fix
that)

+    virtual views::View* GetAnchorView() = 0;
+    virtual gfx::Point GetAnchorPoint() = 0;
+    virtual gfx::NativeView GetParentView() = 0;
+  };
+
+  PermissionBubbleViewViews(Browser* browser,
+                            std::unique_ptr<AnchorDelegate> anchor_delegate);

[tapted, 2016/05/03 12:02:39]

Since both AnchorDelegates are just created from the Browser* I think we can
avoid having to pass this in, and just initialize anchor_delegate_ with the
CreateAnchorDelegate(browser) function I mentioned earlier

[Elly Fong-Jones, 2016/05/10 21:21:02]

Done.

   ~PermissionBubbleViewViews() override;
 
   // PermissionBubbleView:
   void SetDelegate(Delegate* delegate) override;
   void Show(const std::vector<PermissionBubbleRequest*>& requests,
             const std::vector<bool>& accept_state) override;
   bool CanAcceptRequestUpdate() override;
   void Hide() override;
   bool IsVisible() override;
   void UpdateAnchorPosition() override;
   gfx::NativeWindow GetNativeWindow() override;
 
   void Closing();
   void Toggle(int index, bool value);
   void Accept();
   void Deny();
 
  private:
   views::View* GetAnchorView();
+  gfx::Point GetAnchorPoint();

[tapted, 2016/05/03 12:02:39]

this might not be needed. In fact we can probably remove GetAnchorView as well,
since it's just anchor_delegate->GetAnchorView() now and there are just 1-2 call
sites (depending how much we can lean on UpdateAnchor())

[Elly Fong-Jones, 2016/05/10 21:21:02]

Done.

   views::BubbleBorder::Arrow GetAnchorArrow();
 
   Browser* browser_;
   Delegate* delegate_;
   PermissionsBubbleDialogDelegateView* bubble_delegate_;
+  std::unique_ptr<AnchorDelegate> anchor_delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(PermissionBubbleViewViews);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_PERMISSIONS_BUBBLE_VIEW_H_