Update Confirmation UI for saved password change

chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc

 // Copyright 2013 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/passwords/manage_passwords_bubble_view.h"
 
 #include "base/timer/timer.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
@@ skipping 357 matching lines @@
   void StyledLabelLinkClicked(const gfx::Range& range,
                               int event_flags) override;
 
   // Handles the event when the user changes an index of a combobox.
   void OnPerformAction(views::Combobox* source) override;
 
   ManagePasswordsBubbleView* parent_;
 
   views::BlueButton* save_button_;
 
+  views::LabelButton* nope_button_;
+
   // The combobox doesn't take ownership of its model. If we created a
   // combobox we need to ensure that we delete the model here, and because the
   // combobox uses the model in it's destructor, we need to make sure we
   // delete the model _after_ the combobox itself is deleted.
   scoped_ptr<SavePasswordRefusalComboboxModel> combobox_model_;
   scoped_ptr<views::Combobox> refuse_combobox_;
 
   DISALLOW_COPY_AND_ASSIGN(PendingView);
 };
 
 ManagePasswordsBubbleView::PendingView::PendingView(
     ManagePasswordsBubbleView* parent)
-    : parent_(parent) {
+    : parent_(parent), save_button_(nullptr), nope_button_(nullptr) {
   views::GridLayout* layout = new views::GridLayout(this);
   layout->set_minimum_size(gfx::Size(kDesiredBubbleWidth, 0));
   SetLayoutManager(layout);
 
-  std::vector<const autofill::PasswordForm*> credentials(
-      1, &parent->model()->pending_password());
+  std::vector<const autofill::PasswordForm*> credentials;
+  if (parent->model()->account_for_update_selection()) {

[vabr (Chromium), 2015/06/19 18:03:16]

Should you actually create a separate view rather than forking most of the code
for PendingView based on account_for_update_selection()?

(I'm not sure myself, please consult with vasilii@ or mkwst@.)

[dvadym, 2015/06/22 14:43:33]

Yeah, this is a good question that constantly appears, duplication vs more clear
separate logic. We definitely need to discuss it.

+    for (auto local_credential : parent->model()->local_credentials())
+      credentials.push_back(local_credential);
+  } else {
+    credentials.push_back(&parent->model()->pending_password());
+  }
   // Create the pending credential item, save button and refusal combobox.
   ManagePasswordItemsView* item =
       new ManagePasswordItemsView(parent_->model(), credentials);
-  save_button_ = new views::BlueButton(
-      this, l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_SAVE_BUTTON));
-  save_button_->SetFontList(ui::ResourceBundle::GetSharedInstance().GetFontList(
-      ui::ResourceBundle::SmallFont));
-
-  combobox_model_.reset(new SavePasswordRefusalComboboxModel);
-  refuse_combobox_.reset(new views::Combobox(combobox_model_.get()));
-  refuse_combobox_->set_listener(this);
-  refuse_combobox_->SetStyle(views::Combobox::STYLE_ACTION);
-  // TODO(mkwst): Need a mechanism to pipe a font list down into a combobox.
+  if (parent->model()->is_password_change_form_without_username()) {
+    nope_button_ = new views::LabelButton(
+        this, l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_CANCEL_BUTTON));
+    nope_button_->SetStyle(views::Button::STYLE_BUTTON);
+    nope_button_->SetFontList(
+        ui::ResourceBundle::GetSharedInstance().GetFontList(
+            ui::ResourceBundle::SmallFont));
+  } else {
+    combobox_model_.reset(new SavePasswordRefusalComboboxModel);
+    refuse_combobox_.reset(new views::Combobox(combobox_model_.get()));
+    refuse_combobox_->set_listener(this);
+    refuse_combobox_->SetStyle(views::Combobox::STYLE_ACTION);
+    // TODO(mkwst): Need a mechanism to pipe a font list down into a combobox.
+  }
+  if (!parent->model()->account_for_update_selection()) {
+    save_button_ =
+        new views::BlueButton(this, parent_->model()->save_button_text());
+    save_button_->SetFontList(
+        ui::ResourceBundle::GetSharedInstance().GetFontList(
+            ui::ResourceBundle::SmallFont));
+  }
 
   // Title row.
   views::StyledLabel* title_label =
       new views::StyledLabel(parent_->model()->title(), this);
   title_label->SetBaseFontList(
       ui::ResourceBundle::GetSharedInstance().GetFontList(
           ui::ResourceBundle::MediumFont));
   if (!parent_->model()->title_brand_link_range().is_empty()) {
     title_label->AddStyleRange(
         parent_->model()->title_brand_link_range(),
         views::StyledLabel::RangeStyleInfo::CreateForLink());
   }
   BuildColumnSet(layout, SINGLE_VIEW_COLUMN_SET);
   layout->StartRow(0, SINGLE_VIEW_COLUMN_SET);
   layout->AddView(title_label);
   layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
 
   // Credential row.
   layout->StartRow(0, SINGLE_VIEW_COLUMN_SET);
   layout->AddView(item);
 
   // Button row.
-  BuildColumnSet(layout, DOUBLE_BUTTON_COLUMN_SET);
-  layout->StartRowWithPadding(
-      0, DOUBLE_BUTTON_COLUMN_SET, 0, views::kRelatedControlVerticalSpacing);
-  layout->AddView(save_button_);
-  layout->AddView(refuse_combobox_.get());
+  ColumnSetType button_column_type =
+      parent->model()->account_for_update_selection()
+          ? SINGLE_BUTTON_COLUMN_SET
+          : DOUBLE_BUTTON_COLUMN_SET;
+  BuildColumnSet(layout, button_column_type);
+  layout->StartRowWithPadding(0, button_column_type, 0,
+                              views::kRelatedControlVerticalSpacing);
+  if (parent->model()->account_for_update_selection()) {
+    layout->AddView(nope_button_);
+  } else {
+    layout->AddView(save_button_);
+    if (parent->model()->is_password_change_form_without_username())
+      layout->AddView(nope_button_);
+    else
+      layout->AddView(refuse_combobox_.get());
+  }
 
   // Extra padding for visual awesomeness.
   layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
 
   parent_->set_initially_focused_view(save_button_);
 }
 
 ManagePasswordsBubbleView::PendingView::~PendingView() {
 }
 
 void ManagePasswordsBubbleView::PendingView::ButtonPressed(
     views::Button* sender,
     const ui::Event& event) {
-  DCHECK(sender == save_button_);
-  parent_->model()->OnSaveClicked();
+  DCHECK(sender == save_button_ || sender == nope_button_);
+  if (sender == save_button_)
+    parent_->model()->OnSaveClicked();
   parent_->Close();
 }
 
 void ManagePasswordsBubbleView::PendingView::StyledLabelLinkClicked(
     const gfx::Range& range,
     int event_flags) {
   DCHECK_EQ(range, parent_->model()->title_brand_link_range());
   parent_->model()->OnBrandLinkClicked();
 }
 
@@ skipping 755 matching lines @@
 void ManagePasswordsBubbleView::NotifyNeverForThisSiteClicked() {
   if (model()->local_credentials().empty()) {
     // Skip confirmation if there are no existing passwords for this site.
     NotifyConfirmedNeverForThisSite();
   } else {
     model()->OnConfirmationForNeverForThisSite();
     Refresh();
     SizeToContents();
   }
 }