Simplify GetLoginsRequest, and let PasswordStore::GetLoginsImpl take scoped_ptr<GetLoginsRequest>

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 049a64167c770a5e93d78e41a2952d6b4d3fc707..a420e9e707b206bb4714f78c015b8beca60d3764 100644
--- a/components/password_manager/core/browser/test_password_store.cc
+++ b/components/password_manager/core/browser/test_password_store.cc
@@ -47,11 +47,12 @@ bool TestPasswordStore::FormsAreEquivalent(const autofill::PasswordForm& lhs,
 
 void TestPasswordStore::GetAutofillableLoginsImpl(
     scoped_ptr<GetLoginsRequest> request) {
+  ScopedVector<autofill::PasswordForm> results;
   for (const auto& forms_for_realm : stored_passwords_) {
     for (const autofill::PasswordForm& form : forms_for_realm.second)
-      request->result()->push_back(new autofill::PasswordForm(form));
+      results.push_back(new autofill::PasswordForm(form));
   }
-  ForwardLoginsResult(request.Pass());
+  request->NotifyConsumerWithResults(results.Pass());
 }
 
 PasswordStoreChangeList TestPasswordStore::AddLoginImpl(