Convert location bar bubble delegates to bubble dialog delegates

chrome/browser/ui/views/autofill/save_card_bubble_views.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/autofill/save_card_bubble_views.h"
 
 #include <stddef.h>
 
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
@@ skipping 14 matching lines @@
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/layout_constants.h"
 
 namespace autofill {
 
 namespace {
 
 // Fixed width of the bubble.
 const int kBubbleWidth = 395;
 
-// TODO(bondd): BubbleManager will eventually move this logic somewhere else,
-// and then kIsOkButtonOnLeftSide can be removed from here and
-// dialog_client_view.cc.
-#if defined(OS_WIN) || defined(OS_CHROMEOS)
-const bool kIsOkButtonOnLeftSide = true;
-#else
-const bool kIsOkButtonOnLeftSide = false;
-#endif
-
 scoped_ptr<views::StyledLabel> CreateLegalMessageLineLabel(
     const LegalMessageLine& line,
     views::StyledLabelListener* listener) {
   scoped_ptr<views::StyledLabel> label(
       new views::StyledLabel(line.text(), listener));
   for (const LegalMessageLine::Link& link : line.links()) {
     label->AddStyleRange(link.range,
                          views::StyledLabel::RangeStyleInfo::CreateForLink());
   }
   return label;
 }
 
 }  // namespace
 
 SaveCardBubbleViews::SaveCardBubbleViews(views::View* anchor_view,
                                          content::WebContents* web_contents,
                                          SaveCardBubbleController* controller)
     : LocationBarBubbleDelegateView(anchor_view, web_contents),
       controller_(controller),
-      save_button_(nullptr),
-      cancel_button_(nullptr),
       learn_more_link_(nullptr) {
   DCHECK(controller);
-  views::BubbleDelegateView::CreateBubble(this);
+  views::BubbleDialogDelegateView::CreateBubble(this);
 }
 
 SaveCardBubbleViews::~SaveCardBubbleViews() {}
 
 void SaveCardBubbleViews::Show(DisplayReason reason) {
   ShowForReason(reason);
 }
 
 void SaveCardBubbleViews::Hide() {
   controller_ = nullptr;
   Close();
 }
 
+views::View* SaveCardBubbleViews::CreateExtraView() {
+  if (!learn_more_link_) {
+    learn_more_link_ =
+        new views::Link(l10n_util::GetStringUTF16(IDS_LEARN_MORE));
+    learn_more_link_->SetUnderline(false);
+    learn_more_link_->set_listener(this);
+  }
+  return learn_more_link_;
+}
+
 views::View* SaveCardBubbleViews::CreateFootnoteView() {
   if (controller_->GetLegalMessageLines().empty())
     return nullptr;
 
   // Use BoxLayout to provide insets around the label.
   View* view = new View();
   view->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
 
   // Add a StyledLabel for each line of the legal message.
   for (const LegalMessageLine& line : controller_->GetLegalMessageLines())
     view->AddChildView(CreateLegalMessageLineLabel(line, this).release());
 
   return view;
 }
 
+bool SaveCardBubbleViews::Accept() {
+  controller_->OnSaveButton();
+  return true;

[msw, 2016/03/03 23:01:39]

nit: maybe return LocationBarBubbleDelegateView::Accept here and elsewhere?

[Evan Stade, 2016/03/07 18:36:38]

I don't think that's right. This seems like a situation where you want to
override default (non-) behavior, not tack on extra behavior.

+}
+
+bool SaveCardBubbleViews::Cancel() {
+  controller_->OnCancelButton();
+  return true;

[msw, 2016/03/03 23:01:39]

nit: maybe return LocationBarBubbleDelegateView::Cancel here and elsewhere?

[Evan Stade, 2016/03/07 18:36:38]

ditto

+}
+
+int SaveCardBubbleViews::GetDialogButtons() const {
+  // This is the default for BubbleDialogDelegateView, but it's not the default
+  // for LocationBarBubbleDelegateView.
+  return ui::DIALOG_BUTTON_OK | ui::DIALOG_BUTTON_CANCEL;
+}
+
+base::string16 SaveCardBubbleViews::GetDialogButtonLabel(
+    ui::DialogButton button) const {
+  return l10n_util::GetStringUTF16(button == ui::DIALOG_BUTTON_OK
+                                       ? IDS_AUTOFILL_SAVE_CARD_PROMPT_ACCEPT
+                                       : IDS_AUTOFILL_SAVE_CARD_PROMPT_DENY);
+}
+
+bool SaveCardBubbleViews::ShouldDefaultButtonBeBlue() const {
+  return true;
+}
+
 gfx::Size SaveCardBubbleViews::GetPreferredSize() const {
   return gfx::Size(kBubbleWidth, GetHeightForWidth(kBubbleWidth));
 }
 
-views::View* SaveCardBubbleViews::GetInitiallyFocusedView() {
-  return save_button_;
-}
-
 base::string16 SaveCardBubbleViews::GetWindowTitle() const {
   return controller_->GetWindowTitle();
 }
 
 void SaveCardBubbleViews::WindowClosing() {
   if (controller_)
     controller_->OnBubbleClosed();
 }
 
-void SaveCardBubbleViews::ButtonPressed(views::Button* sender,
-                                        const ui::Event& event) {
-  if (sender == save_button_) {
-    controller_->OnSaveButton();
-  } else {
-    DCHECK_EQ(sender, cancel_button_);
-    controller_->OnCancelButton();
-  }
-  Close();
-}
-
 void SaveCardBubbleViews::LinkClicked(views::Link* source, int event_flags) {
   DCHECK_EQ(source, learn_more_link_);
   controller_->OnLearnMoreClicked();
 }
 
 void SaveCardBubbleViews::StyledLabelLinkClicked(views::StyledLabel* label,
                                                  const gfx::Range& range,
                                                  int event_flags) {
   // Index of |label| within its parent's view hierarchy is the same as the
   // legal message line index. DCHECK this assumption to guard against future
@@ skipping 47 matching lines @@
 
   // Optionally add label that will contain an explanation for upload.
   base::string16 explanation = controller_->GetExplanatoryMessage();
   if (!explanation.empty()) {
     views::Label* explanation_label = new views::Label(explanation);
     explanation_label->SetMultiLine(true);
     explanation_label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
     view->AddChildView(explanation_label);
   }
 
-  // Add "learn more" link and accept/cancel buttons.
-  views::View* button_view = new views::View();
-  views::BoxLayout* button_view_layout = new views::BoxLayout(
-      views::BoxLayout::kHorizontal, 0, 0, views::kRelatedButtonHSpacing);
-  button_view->SetLayoutManager(button_view_layout);
-  view->AddChildView(button_view);
-
-  learn_more_link_ = new views::Link(l10n_util::GetStringUTF16(IDS_LEARN_MORE));
-  learn_more_link_->SetHorizontalAlignment(gfx::ALIGN_LEFT);
-  learn_more_link_->SetUnderline(false);
-  learn_more_link_->set_listener(this);
-  button_view->AddChildView(learn_more_link_);
-  button_view_layout->SetFlexForView(learn_more_link_, 1);
-
-  save_button_ = new views::BlueButton(
-      this, l10n_util::GetStringUTF16(IDS_AUTOFILL_SAVE_CARD_PROMPT_ACCEPT));
-  save_button_->SetIsDefault(true);
-
-  cancel_button_ = new views::LabelButton(
-      this, l10n_util::GetStringUTF16(IDS_AUTOFILL_SAVE_CARD_PROMPT_DENY));
-  cancel_button_->SetStyle(views::Button::STYLE_BUTTON);
-
-  if (kIsOkButtonOnLeftSide) {
-    button_view->AddChildView(save_button_);
-    button_view->AddChildView(cancel_button_);
-  } else {
-    button_view->AddChildView(cancel_button_);
-    button_view->AddChildView(save_button_);
-  }
-
   return view;
 }
 
 void SaveCardBubbleViews::Init() {
   SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
   AddChildView(CreateMainContentView().release());
 }
 
 }  // namespace autofill