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

Unified Diff: chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc

Issue 1009953002: Integrate ManagePasswordsState into ManagePasswordsUIController. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix android Created 5 years, 9 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/passwords/manage_passwords_bubble_view.cc
diff --git a/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc b/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc
index 4992b681e1624cfe75bebf04463d0a6abea39233..8fbc3876b1662842a2ba209025e283eb171c09cb 100644
--- a/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc
+++ b/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc
@@ -163,7 +163,7 @@ class ManagePasswordsBubbleView::AccountChooserView
// Adds |password_forms| to the layout remembering their |type|.
void AddCredentialItemsWithType(
views::GridLayout* layout,
- const ScopedVector<autofill::PasswordForm>& password_forms,
+ const ScopedVector<const autofill::PasswordForm>& password_forms,
password_manager::CredentialType type);
ManagePasswordsBubbleView* parent_;
@@ -188,11 +188,11 @@ ManagePasswordsBubbleView::AccountChooserView::AccountChooserView(
AddTitleRow(layout, parent_->model());
AddCredentialItemsWithType(
- layout, parent_->model()->local_pending_credentials(),
+ layout, parent_->model()->local_credentials(),
password_manager::CredentialType::CREDENTIAL_TYPE_LOCAL);
AddCredentialItemsWithType(
- layout, parent_->model()->federated_pending_credentials(),
+ layout, parent_->model()->federated_credentials(),
password_manager::CredentialType::CREDENTIAL_TYPE_FEDERATED);
// Button row.
@@ -212,11 +212,11 @@ ManagePasswordsBubbleView::AccountChooserView::~AccountChooserView() {
void ManagePasswordsBubbleView::AccountChooserView::AddCredentialItemsWithType(
views::GridLayout* layout,
- const ScopedVector<autofill::PasswordForm>& password_forms,
+ const ScopedVector<const autofill::PasswordForm>& password_forms,
password_manager::CredentialType type) {
net::URLRequestContextGetter* request_context =
parent_->model()->GetProfile()->GetRequestContext();
- for (autofill::PasswordForm* form : password_forms) {
+ for (const autofill::PasswordForm* form : password_forms) {
// Add the title to the layout with appropriate padding.
layout->StartRow(0, SINGLE_VIEW_COLUMN_SET);
layout->AddView(new CredentialsItemView(
@@ -511,13 +511,9 @@ ManagePasswordsBubbleView::ManageView::ManageView(
// If we have a list of passwords to store for the current site, display
// them to the user for management. Otherwise, render a "No passwords for
// this site" message.
- if (!parent_->model()->best_matches().empty()) {
- std::vector<const autofill::PasswordForm*> password_forms;
- for (auto password_form : parent_->model()->best_matches()) {
- password_forms.push_back(password_form.second);
- }
+ if (!parent_->model()->local_credentials().empty()) {
ManagePasswordItemsView* item = new ManagePasswordItemsView(
- parent_->model(), password_forms);
+ parent_->model(), parent_->model()->local_credentials().get());
layout->StartRow(0, SINGLE_VIEW_COLUMN_SET);
layout->AddView(item);
} else {
@@ -613,9 +609,9 @@ ManagePasswordsBubbleView::ManageAccountsView::ManageAccountsView(
BuildColumnSet(layout, SINGLE_VIEW_COLUMN_SET);
AddTitleRow(layout, parent_->model());
- if (!parent_->model()->local_pending_credentials().empty()) {
+ if (!parent_->model()->local_credentials().empty()) {
for (const autofill::PasswordForm* form :
- parent_->model()->local_pending_credentials()) {
+ parent_->model()->local_credentials()) {
layout->StartRow(0, SINGLE_VIEW_COLUMN_SET);
layout->AddView(new ManageCredentialItemView(parent_->model(), form));
}
@@ -1017,7 +1013,7 @@ void ManagePasswordsBubbleView::NotifyUndoNeverForThisSite() {
}
void ManagePasswordsBubbleView::NotifyNeverForThisSiteClicked() {
- if (model()->best_matches().empty()) {
+ if (model()->local_credentials().empty()) {
// Skip confirmation if there are no existing passwords for this site.
NotifyConfirmedNeverForThisSite();
} else {

Powered by Google App Engine
This is Rietveld 408576698