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

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

Issue 2529093002: Hide PSL matches in the password bubble after the account chooser. (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/passwords/manage_password_items_view.cc
diff --git a/chrome/browser/ui/views/passwords/manage_password_items_view.cc b/chrome/browser/ui/views/passwords/manage_password_items_view.cc
index 6a9ddf81b3935bebbf9215bd0fedcc8e98ba656e..8443799ace65850a97f9c961c84770ea61e63eef 100644
--- a/chrome/browser/ui/views/passwords/manage_password_items_view.cc
+++ b/chrome/browser/ui/views/passwords/manage_password_items_view.cc
@@ -271,9 +271,8 @@ ManagePasswordItemsView::ManagePasswordItemsView(
: model_(manage_passwords_bubble_model) {
int fixed_height = PasswordFormRow::GetFixedHeight(model_->state());
for (const auto& password_form : *password_forms) {
- if (!password_form.is_public_suffix_match)
vabr (Chromium) 2016/11/25 15:07:26 What about the following situation: The user save
vasilii 2016/11/25 15:25:18 To be fixed in another place because it's platform
- password_forms_rows_.push_back(base::MakeUnique<PasswordFormRow>(
- this, &password_form, fixed_height));
+ password_forms_rows_.push_back(base::MakeUnique<PasswordFormRow>(
+ this, &password_form, fixed_height));
}
AddRows();
}
« no previous file with comments | « chrome/browser/ui/passwords/manage_passwords_ui_controller_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698