Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(266)

Unified Diff: ui/views/bubble/bubble_dialog_delegate.h

Issue 2908963002: Use buttons from DialogClientView in BookmakBubbleView. (Closed)
Patch Set: Respond to comments Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/views/bubble/bubble_dialog_delegate.h
diff --git a/ui/views/bubble/bubble_dialog_delegate.h b/ui/views/bubble/bubble_dialog_delegate.h
index 81a3a4ca9d8bff20f9c15c2629c7d9873fc99fc1..30be96f44bb759628c590487585d036433d7906b 100644
--- a/ui/views/bubble/bubble_dialog_delegate.h
+++ b/ui/views/bubble/bubble_dialog_delegate.h
@@ -144,6 +144,10 @@ class VIEWS_EXPORT BubbleDialogDelegateView : public DialogDelegateView,
void SetAnchorView(View* anchor_view);
void SetAnchorRect(const gfx::Rect& rect);
+ void set_button_row_insets(const gfx::Insets& insets) {
+ button_row_insets_ = insets;
+ }
+
// Resize and potentially move the bubble to fit the content's preferred size.
void SizeToContents();
@@ -187,7 +191,8 @@ class VIEWS_EXPORT BubbleDialogDelegateView : public DialogDelegateView,
SkColor color_;
bool color_explicitly_set_;
- // The margins between the content and the inside of the border.
+ // The margins between the content (including buttons) and the inside of the
+ // border.
gfx::Insets margins_;
// The margins around the title.
@@ -197,6 +202,10 @@ class VIEWS_EXPORT BubbleDialogDelegateView : public DialogDelegateView,
// Insets applied to the |anchor_view_| bounds.
gfx::Insets anchor_view_insets_;
+ // Insets around the button row (when buttons are present). Note that bottom
+ // and side insets from |margins_| are already added around the button row.
+ gfx::Insets button_row_insets_;
+
// Specifies whether the bubble (or its border) handles mouse events, etc.
bool accept_events_;

Powered by Google App Engine
This is Rietveld 408576698