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

Unified Diff: chrome/browser/ui/passwords/manage_passwords_state.cc

Issue 1402803002: [Password Manager] Pass origin of the form which was autofilled in order to display it in managemen… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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/passwords/manage_passwords_state.cc
diff --git a/chrome/browser/ui/passwords/manage_passwords_state.cc b/chrome/browser/ui/passwords/manage_passwords_state.cc
index 33649b86676a31a67d73d5466eb43c0e97c09939..c8f70c75d529a7be3870bfdc6a912f7424f5f9ec 100644
--- a/chrome/browser/ui/passwords/manage_passwords_state.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_state.cc
@@ -145,7 +145,8 @@ void ManagePasswordsState::OnAutomaticPasswordSave(
}
void ManagePasswordsState::OnPasswordAutofilled(
- const PasswordFormMap& password_form_map) {
+ const PasswordFormMap& password_form_map,
+ const GURL& origin) {
// TODO(vabr): Revert back to DCHECK once http://crbug.com/486931 is fixed.
CHECK(!password_form_map.empty());
ClearData();
@@ -156,7 +157,7 @@ void ManagePasswordsState::OnPasswordAutofilled(
SetState(password_manager::ui::INACTIVE_STATE);
} else {
local_credentials_forms_ = DeepCopyMapToVector(password_form_map);
- origin_ = local_credentials_forms_.front()->origin;
+ origin_ = origin;
SetState(password_manager::ui::MANAGE_STATE);
}
}
« no previous file with comments | « chrome/browser/ui/passwords/manage_passwords_state.h ('k') | chrome/browser/ui/passwords/manage_passwords_state_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698