Views Bubble API adjustments and cleanup

views/bubble/bubble_delegate.h

 // Copyright (c) 2011 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 VIEWS_BUBBLE_BUBBLE_DELEGATE_H_
 #define VIEWS_BUBBLE_BUBBLE_DELEGATE_H_
 #pragma once
 
-#include <string>
-
 #include "views/bubble/bubble_border.h"
-#include "views/view.h"
 #include "views/widget/widget_delegate.h"
 
 namespace views {
-class BubbleBorder;
+
 class BubbleFrameView;
 class BubbleView;
-class View;
 
-// BubbleDelegate interface to create bubble frame view and bubble client view.
+// BubbleDelegateView creates frame and client views for bubble Widgets.
+// BubbleDelegateView itself is the client's contents view.
 //
 ///////////////////////////////////////////////////////////////////////////////
-class VIEWS_EXPORT BubbleDelegate : public WidgetDelegate {
+class VIEWS_EXPORT BubbleDelegateView : public WidgetDelegateView {
  public:
-  virtual BubbleDelegate* AsBubbleDelegate() OVERRIDE;
+  virtual ~BubbleDelegateView();
+
+  // Construct a Bubble Widget from the argument BubbleDelegateView.
+  static Widget* ConstructBubble(BubbleDelegateView* bubble_delegate,
+                                 Widget* parent_widget);
+
+  // WidgetDelegate overrides:
+  virtual View* GetContentsView() OVERRIDE { return this; }
   virtual ClientView* CreateClientView(Widget* widget) OVERRIDE;
   virtual NonClientFrameView* CreateNonClientFrameView() OVERRIDE;
 
-  virtual SkColor GetFrameBackgroundColor() = 0;
-  virtual gfx::Rect GetBounds() = 0;
-  virtual BubbleBorder::ArrowLocation GetFrameArrowLocation() = 0;
+  // Override Init to perform Contents View initialization for bubble sizing.
+  virtual void Init() {}
 
-  const BubbleView* GetBubbleView() const;
+  // Get the arrow's target point in screen space.
+  virtual gfx::Point GetArrowPoint() const;

[alicet1, 2011/10/14 00:21:17]

Is this the same as the anchor point? if so can it be called anchor instead?

+
+  // Get the arrow's location on the bubble.
+  virtual BubbleBorder::ArrowLocation GetArrowLocation() const {
+    return BubbleBorder::TOP_LEFT;
+  }
+
+  // Get the color used for the background and border.
+  virtual SkColor GetColor() const { return SK_ColorWHITE; }
+
   BubbleView* GetBubbleView();
-
-  const BubbleFrameView* GetBubbleFrameView() const;
   BubbleFrameView* GetBubbleFrameView();
-
- protected:
-  virtual ~BubbleDelegate() {}
-};
-
-// BubbleDelegateView to create bubble frame view and bubble client view.
-//
-///////////////////////////////////////////////////////////////////////////////
-class VIEWS_EXPORT BubbleDelegateView : public BubbleDelegate, public View {
- public:
-  explicit BubbleDelegateView(Widget* frame);
-  virtual ~BubbleDelegateView();
-
-  // Overridden from WidgetDelegate:
-  virtual Widget* GetWidget() OVERRIDE;
-  virtual const Widget* GetWidget() const OVERRIDE;
-
- private:
-  Widget* frame_;
-  DISALLOW_COPY_AND_ASSIGN(BubbleDelegateView);
 };
 
 }  // namespace views
 
 #endif  // VIEWS_BUBBLE_BUBBLE_DELEGATE_H_