Commit a48db565 authored by Mohamed Amir Yosef's avatar Mohamed Amir Yosef Committed by Commit Bot

[Passwords] Init PasswordSaveUpdate*View members in the header

Bug: 1044038
Change-Id: I48ac13f6ffc6f44f990c5dfd7f3248ab3b614d85
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2248629Reviewed-by: default avatarVasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#779336}
parent 16460913
...@@ -256,11 +256,6 @@ PasswordSaveUpdateView::PasswordSaveUpdateView( ...@@ -256,11 +256,6 @@ PasswordSaveUpdateView::PasswordSaveUpdateView(
: PasswordBubbleControllerBase::DisplayReason::kUserAction), : PasswordBubbleControllerBase::DisplayReason::kUserAction),
is_update_bubble_(controller_.state() == is_update_bubble_(controller_.state() ==
password_manager::ui::PENDING_PASSWORD_UPDATE_STATE), password_manager::ui::PENDING_PASSWORD_UPDATE_STATE),
sign_in_promo_(nullptr),
username_dropdown_(nullptr),
password_view_button_(nullptr),
password_dropdown_(nullptr),
are_passwords_revealed_( are_passwords_revealed_(
controller_.are_passwords_revealed_when_bubble_is_opened()) { controller_.are_passwords_revealed_when_bubble_is_opened()) {
DCHECK(controller_.state() == password_manager::ui::PENDING_PASSWORD_STATE || DCHECK(controller_.state() == password_manager::ui::PENDING_PASSWORD_STATE ||
......
...@@ -74,13 +74,13 @@ class PasswordSaveUpdateView : public PasswordBubbleViewBase, ...@@ -74,13 +74,13 @@ class PasswordSaveUpdateView : public PasswordBubbleViewBase,
// Different promo dialogs that helps the user get access to credentials // Different promo dialogs that helps the user get access to credentials
// across devices. One of these are non-null when the promotion dialog is // across devices. One of these are non-null when the promotion dialog is
// active. // active.
PasswordSignInPromoView* sign_in_promo_; PasswordSignInPromoView* sign_in_promo_ = nullptr;
views::EditableCombobox* username_dropdown_; views::EditableCombobox* username_dropdown_ = nullptr;
views::ToggleImageButton* password_view_button_; views::ToggleImageButton* password_view_button_ = nullptr;
// The view for the password value. // The view for the password value.
views::EditableCombobox* password_dropdown_; views::EditableCombobox* password_dropdown_ = nullptr;
bool are_passwords_revealed_; bool are_passwords_revealed_;
}; };
......
...@@ -345,10 +345,6 @@ PasswordSaveUpdateWithAccountStoreView::PasswordSaveUpdateWithAccountStoreView( ...@@ -345,10 +345,6 @@ PasswordSaveUpdateWithAccountStoreView::PasswordSaveUpdateWithAccountStoreView(
: PasswordBubbleControllerBase::DisplayReason::kUserAction), : PasswordBubbleControllerBase::DisplayReason::kUserAction),
is_update_bubble_(controller_.state() == is_update_bubble_(controller_.state() ==
password_manager::ui::PENDING_PASSWORD_UPDATE_STATE), password_manager::ui::PENDING_PASSWORD_UPDATE_STATE),
username_dropdown_(nullptr),
password_view_button_(nullptr),
password_dropdown_(nullptr),
are_passwords_revealed_( are_passwords_revealed_(
controller_.are_passwords_revealed_when_bubble_is_opened()) { controller_.are_passwords_revealed_when_bubble_is_opened()) {
DCHECK(controller_.state() == password_manager::ui::PENDING_PASSWORD_STATE || DCHECK(controller_.state() == password_manager::ui::PENDING_PASSWORD_STATE ||
......
...@@ -103,11 +103,11 @@ class PasswordSaveUpdateWithAccountStoreView ...@@ -103,11 +103,11 @@ class PasswordSaveUpdateWithAccountStoreView
views::Combobox* destination_dropdown_ = nullptr; views::Combobox* destination_dropdown_ = nullptr;
views::EditableCombobox* username_dropdown_; views::EditableCombobox* username_dropdown_ = nullptr;
views::ToggleImageButton* password_view_button_; views::ToggleImageButton* password_view_button_ = nullptr;
// The view for the password value. // The view for the password value.
views::EditableCombobox* password_dropdown_; views::EditableCombobox* password_dropdown_ = nullptr;
bool are_passwords_revealed_; bool are_passwords_revealed_;
feature_engagement::Tracker* iph_tracker_ = nullptr; feature_engagement::Tracker* iph_tracker_ = nullptr;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment