Index: components/password_manager/content/browser/credential_manager_dispatcher_unittest.cc |
diff --git a/components/password_manager/content/browser/credential_manager_dispatcher_unittest.cc b/components/password_manager/content/browser/credential_manager_dispatcher_unittest.cc |
index 46bbccc3090306b4f9d1ed3001b9089f6487cd04..a6512024d8122cbd783b22ed986cf3d009a6525b 100644 |
--- a/components/password_manager/content/browser/credential_manager_dispatcher_unittest.cc |
+++ b/components/password_manager/content/browser/credential_manager_dispatcher_unittest.cc |
@@ -59,21 +59,14 @@ class TestPasswordManagerClient |
} |
bool PromptUserToChooseCredentials( |
- const std::vector<autofill::PasswordForm*>& local_forms, |
- const std::vector<autofill::PasswordForm*>& federated_forms, |
+ ScopedVector<autofill::PasswordForm> local_forms, |
+ ScopedVector<autofill::PasswordForm> federated_forms, |
base::Callback<void(const password_manager::CredentialInfo&)> callback) |
override { |
- // TODO(melandory): Use ScopedVector instead of std::vector in arguments. |
- // ContentCredentialManagerDispatcher::OnGetPasswordStoreResults contains a |
- // memory leak because of this. |
EXPECT_FALSE(local_forms.empty() && federated_forms.empty()); |
did_prompt_user_to_choose_ = true; |
- ScopedVector<autofill::PasswordForm> local_entries; |
- local_entries.assign(local_forms.begin(), local_forms.end()); |
- ScopedVector<autofill::PasswordForm> federated_entries; |
- federated_entries.assign(federated_forms.begin(), federated_forms.end()); |
password_manager::CredentialInfo info( |
- local_forms.empty() ? *federated_forms[0] : *local_entries[0], |
+ local_forms.empty() ? *federated_forms[0] : *local_forms[0], |
local_forms.empty() |
? password_manager::CredentialType::CREDENTIAL_TYPE_FEDERATED |
: password_manager::CredentialType::CREDENTIAL_TYPE_LOCAL); |