Chromium Code Reviews| Index: ui/message_center/message_bubble_base.h |
| diff --git a/ui/message_center/message_bubble_base.h b/ui/message_center/message_bubble_base.h |
| index 9946ada3319057a4cf56948de68f3d382ba01c7f..8a1f9419630b7f0428fd0769f6aebad65e7907e2 100644 |
| --- a/ui/message_center/message_bubble_base.h |
| +++ b/ui/message_center/message_bubble_base.h |
| @@ -6,6 +6,7 @@ |
| #define UI_MESSAGE_CENTER_MESSAGE_BUBBLE_BASE_H_ |
| #include "base/memory/scoped_ptr.h" |
| +#include "ui/message_center/message_center.h" |
| #include "ui/message_center/message_center_export.h" |
| #include "ui/message_center/notification_list.h" |
|
Jun Mukai
2013/03/01 21:44:42
safe to remove this inclusion?
dharcourt
2013/03/02 02:08:02
Yes, thanks for catching this. Done.
|
| #include "ui/views/bubble/tray_bubble_view.h" |
| @@ -14,7 +15,7 @@ namespace message_center { |
| class MESSAGE_CENTER_EXPORT MessageBubbleBase { |
| public: |
| - explicit MessageBubbleBase(NotificationList::Delegate* list_delegate); |
| + explicit MessageBubbleBase(MessageCenter* message_center); |
|
dharcourt
2013/03/01 10:17:31
Because NotificationList::Delegate doesn't provide
|
| virtual ~MessageBubbleBase(); |
| @@ -61,13 +62,13 @@ class MESSAGE_CENTER_EXPORT MessageBubbleBase { |
| protected: |
| views::TrayBubbleView::InitParams GetDefaultInitParams( |
| views::TrayBubbleView::AnchorAlignment anchor_alignment); |
| - NotificationList::Delegate* list_delegate() { return list_delegate_; } |
| + MessageCenter* message_center() { return message_center_; } |
| void set_bubble_view(views::TrayBubbleView* bubble_view) { |
| bubble_view_ = bubble_view; |
| } |
| private: |
| - NotificationList::Delegate* list_delegate_; |
| + MessageCenter* message_center_; |
| views::TrayBubbleView* bubble_view_; |
| base::WeakPtrFactory<MessageBubbleBase> weak_ptr_factory_; |
| int max_height_; |