Edit button makes username editable in the password manager bubble.

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/macros.h"
 #include "base/metrics/user_metrics.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/timer/timer.h"
@@ skipping 46 matching lines @@
 // Helpers --------------------------------------------------------------------
 
 namespace {
 
 enum ColumnSetType {
   // | | (FILL, FILL) | |
   // Used for the bubble's header, the credentials list, and for simple
   // messages like "No passwords".
   SINGLE_VIEW_COLUMN_SET,
 
+  // | | (FILL, FILL) | | (FILL, FILL) | |
+  // Used for the credentials line of the bubble, for the pending view.
+  DOUBLE_VIEW_COLUMN_SET,
+
   // | | (TRAILING, CENTER) | | (TRAILING, CENTER) | |
   // Used for buttons at the bottom of the bubble which should nest at the
   // bottom-right corner.
   DOUBLE_BUTTON_COLUMN_SET,
 
   // | | (LEADING, CENTER) | | (TRAILING, CENTER) | |
   // Used for buttons at the bottom of the bubble which should occupy
   // the corners.
   LINK_BUTTON_COLUMN_SET,
 
   // | | (TRAILING, CENTER) | |
   // Used when there is only one button which should next at the bottom-right
   // corner.
   SINGLE_BUTTON_COLUMN_SET,
 
   // | | (LEADING, CENTER) | | (TRAILING, CENTER) | | (TRAILING, CENTER) | |
   // Used when there are three buttons.
   TRIPLE_BUTTON_COLUMN_SET,
 };
 
 enum TextRowType { ROW_SINGLE, ROW_MULTILINE };
 
 // Construct an appropriate ColumnSet for the given |type|, and add it
 // to |layout|.
 void BuildColumnSet(views::GridLayout* layout, ColumnSetType type) {
   views::ColumnSet* column_set = layout->AddColumnSet(type);
   int full_width = ManagePasswordsBubbleView::kDesiredBubbleWidth;
   const int button_divider = ChromeLayoutProvider::Get()->GetDistanceMetric(
       views::DISTANCE_RELATED_BUTTON_HORIZONTAL);
+  const int column_divider = ChromeLayoutProvider::Get()->GetDistanceMetric(
+      views::DISTANCE_RELATED_CONTROL_HORIZONTAL);
   switch (type) {
     case SINGLE_VIEW_COLUMN_SET:
       column_set->AddColumn(views::GridLayout::FILL,
                             views::GridLayout::FILL,
                             0,
                             views::GridLayout::FIXED,
                             full_width,
                             0);
       break;
+    case DOUBLE_VIEW_COLUMN_SET:
+      column_set->AddColumn(views::GridLayout::FILL, views::GridLayout::FILL, 1,
+                            views::GridLayout::USE_PREF, 0, 0);
+      column_set->AddPaddingColumn(0, column_divider);
+      column_set->AddColumn(views::GridLayout::FILL, views::GridLayout::FILL, 1,
+                            views::GridLayout::USE_PREF, 0, 0);
+      break;
     case DOUBLE_BUTTON_COLUMN_SET:
       column_set->AddColumn(views::GridLayout::TRAILING,
                             views::GridLayout::CENTER,
                             1,
                             views::GridLayout::USE_PREF,
                             0,
                             0);
       column_set->AddPaddingColumn(0, button_divider);
       column_set->AddColumn(views::GridLayout::TRAILING,
                             views::GridLayout::CENTER,
@@ skipping 147 matching lines @@
 void ManagePasswordsBubbleView::AutoSigninView::OnTimer() {
   parent_->model()->OnAutoSignInToastTimeout();
   parent_->CloseBubble();
 }
 
 // ManagePasswordsBubbleView::PendingView -------------------------------------
 
 // A view offering the user the ability to save credentials. Contains a
 // single ManagePasswordItemsView, along with a "Save Passwords" button,
 // a "Never" button and an "Edit" button to edit username field.
-class ManagePasswordsBubbleView::PendingView : public views::View,
-                                               public views::ButtonListener {
+class ManagePasswordsBubbleView::PendingView
+    : public views::View,
+      public views::ButtonListener,
+      public views::FocusChangeListener {
  public:
   explicit PendingView(ManagePasswordsBubbleView* parent);
   ~PendingView() override;
 
  private:
+  void CreateAndSetLayout();
   // views::ButtonListener:
   void ButtonPressed(views::Button* sender, const ui::Event& event) override;
+  // views::FocusChangeListener:
+  void OnWillChangeFocus(View* focused_before, View* focused_now) override;
+  void OnDidChangeFocus(View* focused_before, View* focused_now) override;
+  // views::View:
+  bool OnKeyPressed(const ui::KeyEvent& event) override;
 
   ManagePasswordsBubbleView* parent_;
 
   views::Button* edit_button_;
   views::Button* save_button_;
   views::Button* never_button_;
+  views::View* username_field_;
+  views::View* password_field_;
+
+  bool editing_;
 
   DISALLOW_COPY_AND_ASSIGN(PendingView);
 };
 
 ManagePasswordsBubbleView::PendingView::PendingView(
     ManagePasswordsBubbleView* parent)
-    : parent_(parent), edit_button_(nullptr) {
+    : parent_(parent),
+      edit_button_(nullptr),
+      save_button_(nullptr),
+      never_button_(nullptr),
+      username_field_(nullptr),
+      password_field_(nullptr),
+      editing_(false) {
+  CreateAndSetLayout();
+  parent_->set_initially_focused_view(save_button_);
+}
+
+void ManagePasswordsBubbleView::PendingView::CreateAndSetLayout() {
   views::GridLayout* layout = new views::GridLayout(this);
   layout->set_minimum_size(gfx::Size(kDesiredBubbleWidth, 0));
   SetLayoutManager(layout);
 
-  // Create the pending credential item, save button and refusal combobox.
-  ManagePasswordItemsView* item = nullptr;
-  if (!parent->model()->pending_password().username_value.empty()) {
-    item = new ManagePasswordItemsView(parent_->model(),
-                                       &parent->model()->pending_password());
-  }
-  if (base::FeatureList::IsEnabled(
+  // Create the edit, save and never buttons.
+  if (!edit_button_ &&
+      base::FeatureList::IsEnabled(
           password_manager::features::kEnableUsernameCorrection)) {
     edit_button_ = views::MdTextButton::CreateSecondaryUiButton(
         this, l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_EDIT_BUTTON));
   }
-  save_button_ = views::MdTextButton::CreateSecondaryUiBlueButton(
-      this, l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_SAVE_BUTTON));
-  never_button_ = views::MdTextButton::CreateSecondaryUiButton(
-      this,
-      l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_BUBBLE_BLACKLIST_BUTTON));
+  if (!save_button_) {
+    save_button_ = views::MdTextButton::CreateSecondaryUiBlueButton(
+        this, l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_SAVE_BUTTON));
+  }
+  if (!never_button_) {
+    never_button_ = views::MdTextButton::CreateSecondaryUiButton(
+        this, l10n_util::GetStringUTF16(
+                  IDS_PASSWORD_MANAGER_BUBBLE_BLACKLIST_BUTTON));
+  }
 
-  BuildColumnSet(layout, SINGLE_VIEW_COLUMN_SET);
-
-  // Credential row.
-  if (item) {
-    layout->StartRow(0, SINGLE_VIEW_COLUMN_SET);
-    layout->AddView(item);
+  // Credentials row.
+  BuildColumnSet(layout, DOUBLE_VIEW_COLUMN_SET);
+  if (!parent_->model()->pending_password().username_value.empty() ||
+      edit_button_) {
+    layout->StartRow(0, DOUBLE_VIEW_COLUMN_SET);
+    const autofill::PasswordForm* password_form =
+        &parent_->model()->pending_password();
+    if (editing_) {
+      DCHECK_EQ(username_field_, nullptr);

[vasilii, 2017/07/06 11:59:32]

Seems like it's true for both editing_ and !editing_

[irmakk, 2017/07/06 12:53:14]

Done.

+      username_field_ = GenerateUsernameEditable(*password_form).release();
+    } else {
+      username_field_ = GenerateUsernameLabel(*password_form).release();
+    }
+    if (!password_field_) {
+      password_field_ = GeneratePasswordLabel(*password_form).release();
+    }
+    layout->AddView(username_field_);
+    layout->AddView(password_field_);
     layout->AddPaddingRow(0,
                           ChromeLayoutProvider::Get()
                               ->GetInsetsMetric(views::INSETS_DIALOG_CONTENTS)
                               .bottom());
   }
-
   // Button row.
   ColumnSetType column_set_type =
       edit_button_ ? TRIPLE_BUTTON_COLUMN_SET : DOUBLE_BUTTON_COLUMN_SET;
   BuildColumnSet(layout, column_set_type);
   layout->StartRow(0, column_set_type);
   if (column_set_type == TRIPLE_BUTTON_COLUMN_SET) {
     layout->AddView(edit_button_);
   }
   layout->AddView(save_button_);
   layout->AddView(never_button_);
 
-  parent_->set_initially_focused_view(save_button_);
+  layout->Layout(this);

[vasilii, 2017/07/06 11:59:32]

Move it to where you switch the state. There is no point in calling Layout from
the constructor

[irmakk, 2017/07/06 12:53:14]

Done.

 }
 
 ManagePasswordsBubbleView::PendingView::~PendingView() {
 }
 
 void ManagePasswordsBubbleView::PendingView::ButtonPressed(
     views::Button* sender,
     const ui::Event& event) {
   // TODO(https://crbug.com/734965): Implement edit button logic.
   if (sender == edit_button_) {
+    edit_button_->SetEnabled(false);
+    editing_ = true;
+    this->RemoveChildView(username_field_);

[vasilii, 2017/07/06 11:59:32]

Is there a good reason to use "this->"?

[irmakk, 2017/07/06 12:53:13]

Done.

+    username_field_ = nullptr;
+    CreateAndSetLayout();
+    this->GetFocusManager()->SetFocusedView(username_field_);
+    parent_->GetFocusManager()->AddFocusChangeListener(this);

[vasilii, 2017/07/06 11:59:32]

It's confusing because this->GetFocusManager() and parent_->GetFocusManager()
are the same. Thus, just use GetFocusManager().

[irmakk, 2017/07/06 12:53:14]

Oh, i didnt know it was inherited. Removing, thanks.

+    parent_->SizeToContents();
     return;
-  } else if (sender == save_button_) {
+  }
+  if (sender == save_button_) {
     parent_->model()->OnSaveClicked();
     if (parent_->model()->ReplaceToShowPromotionIfNeeded()) {
       parent_->Refresh();
       return;
     }
   } else if (sender == never_button_) {
     parent_->model()->OnNeverForThisSiteClicked();
   } else {
     NOTREACHED();
   }
 
   parent_->CloseBubble();
 }
 
+void ManagePasswordsBubbleView::PendingView::OnWillChangeFocus(
+    View* focused_before,
+    View* focused_now) {
+  // Nothing to do here.
+}
+
+void ManagePasswordsBubbleView::PendingView::OnDidChangeFocus(
+    View* focused_before,
+    View* focused_now) {
+  if (editing_ && focused_before == username_field_) {
+    editing_ = false;
+    this->RemoveChildView(username_field_);
+    CreateAndSetLayout();
+    edit_button_->SetEnabled(true);
+    this->GetFocusManager()->SetFocusedView(save_button_);
+    parent_->SizeToContents();
+    this->GetFocusManager()->RemoveFocusChangeListener(this);
+  }
+}
+
+bool ManagePasswordsBubbleView::PendingView::OnKeyPressed(
+    const ui::KeyEvent& event) {
+  if (editing_ && (event.key_code() == ui::KeyboardCode::VKEY_RETURN ||
+                   event.key_code() == ui::KeyboardCode::VKEY_ESCAPE)) {
+    editing_ = false;
+    this->RemoveChildView(username_field_);
+    username_field_ = nullptr;
+    CreateAndSetLayout();
+    edit_button_->SetEnabled(true);
+    this->GetFocusManager()->SetFocusedView(save_button_);
+    parent_->SizeToContents();
+    this->GetFocusManager()->RemoveFocusChangeListener(this);

[vasilii, 2017/07/06 11:59:32]

This is a copy/paste from OnDidChangeFocus(). There should be one helper method
doing this transition.

[irmakk, 2017/07/06 12:53:14]

Done.

+    return true;
+  }
+  return false;
+}
+
 // ManagePasswordsBubbleView::ManageView --------------------------------------
 
 // A view offering the user a list of their currently saved credentials
 // for the current page, along with a "Manage passwords" link and a
 // "Done" button.
 class ManagePasswordsBubbleView::ManageView : public views::View,
                                               public views::ButtonListener,
                                               public views::LinkListener {
  public:
   explicit ManageView(ManagePasswordsBubbleView* parent);
@@ skipping 504 matching lines @@
   } else if (model_.state() ==
              password_manager::ui::CHROME_DESKTOP_IOS_PROMO_STATE) {
     AddChildView(new DesktopIOSPromotionBubbleView(
         model_.GetProfile(),
         desktop_ios_promotion::PromotionEntryPoint::SAVE_PASSWORD_BUBBLE));
 #endif
   } else {
     AddChildView(new ManageView(this));
   }
 }