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

Unified Diff: chrome/browser/ui/views/autofill/autofill_credit_card_bubble_views.cc

Issue 20871003: Fix BubbleFrameView title and close button patterns. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync and merge; add OVERRIDE mark. Created 7 years, 5 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: chrome/browser/ui/views/autofill/autofill_credit_card_bubble_views.cc
diff --git a/chrome/browser/ui/views/autofill/autofill_credit_card_bubble_views.cc b/chrome/browser/ui/views/autofill/autofill_credit_card_bubble_views.cc
index 4853230741b5055c5964cacc38137eaefe6d97fb..fb2f2388085eae5a57c3167d40080381485a96a0 100644
--- a/chrome/browser/ui/views/autofill/autofill_credit_card_bubble_views.cc
+++ b/chrome/browser/ui/views/autofill/autofill_credit_card_bubble_views.cc
@@ -28,8 +28,6 @@ void AutofillCreditCardBubbleViews::Show() {
// TODO(dbeam): investigate why this steals focus from the web contents.
views::BubbleDelegateView::CreateBubble(this);
- GetBubbleFrameView()->SetTitle(controller_->BubbleTitle());
-
GetWidget()->Show();
SizeToContents();
}
@@ -42,6 +40,10 @@ bool AutofillCreditCardBubbleViews::IsHiding() const {
return GetWidget() && GetWidget()->IsClosed();
}
+string16 AutofillCreditCardBubbleViews::GetWindowTitle() const {
+ return controller_->BubbleTitle();
+}
+
void AutofillCreditCardBubbleViews::Init() {
SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 0,
views::kRelatedControlVerticalSpacing));
« no previous file with comments | « chrome/browser/ui/views/autofill/autofill_credit_card_bubble_views.h ('k') | ui/views/bubble/bubble_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698