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

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

Issue 1717453003: Introduce BubbleDialogDelegateView, which extends DialogDelegateView (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 4 years, 10 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/card_unmask_prompt_views.cc
diff --git a/chrome/browser/ui/views/autofill/card_unmask_prompt_views.cc b/chrome/browser/ui/views/autofill/card_unmask_prompt_views.cc
index e1f951f8543d049e66a5e2b59ff8ddf0995c7487..473980f7ab9ddee3450d4e765a85ef9e8899a3d5 100644
--- a/chrome/browser/ui/views/autofill/card_unmask_prompt_views.cc
+++ b/chrome/browser/ui/views/autofill/card_unmask_prompt_views.cc
@@ -213,7 +213,7 @@ views::View* CardUnmaskPromptViews::GetContentsView() {
return this;
}
-views::View* CardUnmaskPromptViews::CreateFootnoteView() {
+scoped_ptr<views::View> CardUnmaskPromptViews::CreateFootnoteView() {
if (!controller_->CanStoreLocally())
return nullptr;
@@ -236,7 +236,7 @@ views::View* CardUnmaskPromptViews::CreateFootnoteView() {
storage_row_->AddChildView(new TooltipIcon(l10n_util::GetStringUTF16(
IDS_AUTOFILL_CARD_UNMASK_PROMPT_STORAGE_TOOLTIP)));
- return storage_row_;
+ return scoped_ptr<views::View>(storage_row_);
}
gfx::Size CardUnmaskPromptViews::GetPreferredSize() const {
@@ -288,10 +288,6 @@ void CardUnmaskPromptViews::DeleteDelegate() {
delete this;
}
-int CardUnmaskPromptViews::GetDialogButtons() const {
- return ui::DIALOG_BUTTON_OK | ui::DIALOG_BUTTON_CANCEL;
-}
-
base::string16 CardUnmaskPromptViews::GetDialogButtonLabel(
ui::DialogButton button) const {
if (button == ui::DIALOG_BUTTON_OK)

Powered by Google App Engine
This is Rietveld 408576698