Views Bubble API adjustments and cleanup

views/bubble/bubble_delegate.cc

 // 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.
 
 #include "views/bubble/bubble_delegate.h"
+
 #include "views/bubble/bubble_frame_view.h"
 #include "views/bubble/bubble_view.h"
-
-#include "base/logging.h"
 #include "views/widget/widget.h"
 
 namespace views {
 
-BubbleDelegate* BubbleDelegate::AsBubbleDelegate() { return this; }
+BubbleDelegateView::~BubbleDelegateView() {}
 
-ClientView* BubbleDelegate::CreateClientView(Widget* widget) {
-  BubbleView* bubble_view = new BubbleView(widget, GetContentsView());
-  bubble_view->SetBounds(0, 0, GetBounds().width(), GetBounds().height());
-  if (widget->GetFocusManager()) {
-    widget->GetFocusManager()->RegisterAccelerator(
-        views::Accelerator(ui::VKEY_ESCAPE, false, false, false),
-        bubble_view);
-  }
-  return bubble_view;
+// static
+Widget* BubbleDelegateView::ConstructBubble(BubbleDelegateView* bubble_delegate,
+                                            Widget* parent_widget) {
+  bubble_delegate->Init();
+  views::Widget* bubble_widget = new views::Widget();
+  views::Widget::InitParams params(views::Widget::InitParams::TYPE_BUBBLE);
+  params.delegate = bubble_delegate;
+  params.transparent = true;
+  if (!parent_widget)
+    params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
+  params.parent_widget = parent_widget;
+  bubble_widget->Init(params);
+
+  // TODO(msw): Adjust the widget bounds to point the arrow as intended.
+  bubble_widget->SetBounds(gfx::Rect(bubble_delegate->GetArrowPoint(),
+      bubble_delegate->GetBubbleFrameView()->GetPreferredSize()));
+
+  return bubble_widget;
 }
 
-NonClientFrameView* BubbleDelegate::CreateNonClientFrameView() {
-  return new BubbleFrameView(GetWidget(),
-                             GetBounds(),
-                             GetFrameBackgroundColor(),
-                             GetFrameArrowLocation());
+ClientView* BubbleDelegateView::CreateClientView(Widget* widget) {
+  return new BubbleView(widget, GetContentsView());
 }
 
-const BubbleView* BubbleDelegate::GetBubbleView() const {
+NonClientFrameView* BubbleDelegateView::CreateNonClientFrameView() {
+  return new BubbleFrameView(GetArrowLocation(),
+                             GetPreferredSize(),
+                             GetColor());
+}
+
+gfx::Point BubbleDelegateView::GetArrowPoint() const {
+  return gfx::Point();
+}
+
+BubbleView* BubbleDelegateView::GetBubbleView() {
   return GetWidget()->client_view()->AsBubbleView();
 }
 
-BubbleView* BubbleDelegate::GetBubbleView() {
-  return GetWidget()->client_view()->AsBubbleView();
-}
-
-const BubbleFrameView* BubbleDelegate::GetBubbleFrameView() const {
+BubbleFrameView* BubbleDelegateView::GetBubbleFrameView() {
   return static_cast<BubbleFrameView*>(
       GetWidget()->non_client_view()->frame_view());
 }
 
-BubbleFrameView* BubbleDelegate::GetBubbleFrameView() {
-  return static_cast<BubbleFrameView*>(
-      GetWidget()->non_client_view()->frame_view());
-}
-
-BubbleDelegateView::BubbleDelegateView(Widget* frame):frame_(frame) {
-}
-
-BubbleDelegateView::~BubbleDelegateView() {
-}
-
-Widget* BubbleDelegateView::GetWidget() {
-  return frame_;
-}
-
-const Widget* BubbleDelegateView::GetWidget() const {
-  return frame_;
-}
-
 }  // namespace views