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

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

Issue 2581493002: PageInfo bubble: Use the non-client view's window title and close button. (Closed)
Patch Set: Created 4 years 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.cc
diff --git a/ui/views/bubble/bubble_dialog_delegate.cc b/ui/views/bubble/bubble_dialog_delegate.cc
index 5c9d89be7769c3c5103762059a3e145cb599d7d7..640bfd3eeeb7c80822dac429de9ef88fd4a608f0 100644
--- a/ui/views/bubble/bubble_dialog_delegate.cc
+++ b/ui/views/bubble/bubble_dialog_delegate.cc
@@ -102,9 +102,7 @@ ClientView* BubbleDialogDelegateView::CreateClientView(Widget* widget) {
NonClientFrameView* BubbleDialogDelegateView::CreateNonClientFrameView(
Widget* widget) {
- BubbleFrameView* frame = new BubbleFrameView(
- gfx::Insets(kPanelVertMargin, kPanelHorizMargin, 0, kPanelHorizMargin),
- margins());
+ BubbleFrameView* frame = new BubbleFrameView(title_margins(), margins());
// Note: In CreateBubble, the call to SizeToContents() will cause
// the relayout that this call requires.
frame->SetTitleFontList(GetTitleFontList());
@@ -216,6 +214,7 @@ BubbleDialogDelegateView::BubbleDialogDelegateView(View* anchor_view,
kPanelHorizMargin,
kPanelVertMargin,
kPanelHorizMargin),
+ title_margins_(kPanelVertMargin, kPanelHorizMargin, 0, kPanelHorizMargin),
accept_events_(true),
border_accepts_events_(true),
adjust_if_offscreen_(true),
« ui/views/bubble/bubble_dialog_delegate.h ('K') | « ui/views/bubble/bubble_dialog_delegate.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698