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/platform_util.h"

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

nit: is this needed here?

[Elly Fong-Jones, 2016/05/11 19:16:38]

Done.

 #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:
+  // A PermissionBubbleViewViews owns an AnchorDelegate, which is passed in when

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

nit: update comment (e.g. the delegate is no longer passed in)

[Elly Fong-Jones, 2016/05/11 19:16:38]

Done.

+  // 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() {}
+    virtual ~AnchorDelegate() {}

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

(removing all 3 AnchorDelegate destructors should do the trick to make the
compiler happy, and we can remove this)

[Elly Fong-Jones, 2016/05/11 19:16:38]

Done.

+    virtual views::View* GetAnchorView() = 0;
+    virtual gfx::Point GetAnchorPoint() = 0;
+  };
+
   explicit PermissionBubbleViewViews(Browser* browser);
   ~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();
   views::BubbleBorder::Arrow GetAnchorArrow();
 
+  static std::unique_ptr<AnchorDelegate> CreateAnchorDelegate(Browser* browser);

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

nit: needs a comment - (be sure to mention that it's implemented in
platform-specific files)

[Elly Fong-Jones, 2016/05/11 19:16:38]

Done.

+
   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_