PasswordStore: Use ScopedVector to express ownership of forms

components/password_manager/core/browser/test_password_store.cc

Index: components/password_manager/core/browser/test_password_store.cc
diff --git a/components/password_manager/core/browser/test_password_store.cc b/components/password_manager/core/browser/test_password_store.cc
index ce361e5a89121bf1f93eab3ee1c1b29b763edb6b..1645b19b0ed9dc7444d6ac54c3cdd0ad1ffdf93b 100644
--- a/components/password_manager/core/browser/test_password_store.cc
+++ b/components/password_manager/core/browser/test_password_store.cc
@@ -98,14 +98,13 @@ void TestPasswordStore::GetLoginsImpl(
     const autofill::PasswordForm& form,
     PasswordStore::AuthorizationPromptPolicy prompt_policy,
     const PasswordStore::ConsumerCallbackRunner& runner) {
-  std::vector<autofill::PasswordForm*> matched_forms;
+  ScopedVector<autofill::PasswordForm> matched_forms;
   std::vector<autofill::PasswordForm> forms =
       stored_passwords_[form.signon_realm];
-  for (std::vector<autofill::PasswordForm>::iterator it = forms.begin();
-       it != forms.end(); ++it) {
-    matched_forms.push_back(new autofill::PasswordForm(*it));
+  for (const auto& stored_form : forms) {
+    matched_forms.push_back(new autofill::PasswordForm(stored_form));
   }
-  runner.Run(matched_forms);
+  runner.Run(&matched_forms);
 }
 
 PasswordStoreChangeList TestPasswordStore::RemoveLoginsCreatedBetweenImpl(
@@ -123,12 +122,12 @@ PasswordStoreChangeList TestPasswordStore::RemoveLoginsSyncedBetweenImpl(
 }
 
 bool TestPasswordStore::FillAutofillableLogins(
-    std::vector<autofill::PasswordForm*>* forms) {
+    ScopedVector<autofill::PasswordForm>* forms) {
   return true;
 }
 
 bool TestPasswordStore::FillBlacklistLogins(
-    std::vector<autofill::PasswordForm*>* forms) {
+    ScopedVector<autofill::PasswordForm>* forms) {
   return true;
 }