GetLoginsRequest: Use ScopedVector to express ownership of forms

components/password_manager/core/browser/password_store_default.cc

Index: components/password_manager/core/browser/password_store_default.cc
diff --git a/components/password_manager/core/browser/password_store_default.cc b/components/password_manager/core/browser/password_store_default.cc
index ea493e321b3f399a4dbebdd695a2fdf472e59d8c..ff71b87c5332a28e804afd6e95e9ea64ca926856 100644
--- a/components/password_manager/core/browser/password_store_default.cc
+++ b/components/password_manager/core/browser/password_store_default.cc
@@ -122,24 +122,17 @@ void PasswordStoreDefault::GetLoginsImpl(
 }
 
 void PasswordStoreDefault::GetAutofillableLoginsImpl(
-    GetLoginsRequest* request) {
-  // TODO(vabr) -- request should have a ScopedVector<autofill::PasswordForm>
-  // instead of using |logins| here.
+    scoped_ptr<GetLoginsRequest> request) {
   DCHECK(request->result()->empty());
-  ScopedVector<autofill::PasswordForm> logins;
-  FillAutofillableLogins(&logins);
-  logins.swap(*request->result());
-  ForwardLoginsResult(request);
+  FillAutofillableLogins(request->result());
+  ForwardLoginsResult(request.Pass());
 }
 
-void PasswordStoreDefault::GetBlacklistLoginsImpl(GetLoginsRequest* request) {
-  // TODO(vabr) -- request should have a ScopedVector<autofill::PasswordForm>
-  // instead of using |logins| here.
+void PasswordStoreDefault::GetBlacklistLoginsImpl(
+    scoped_ptr<GetLoginsRequest> request) {
   DCHECK(request->result()->empty());
-  ScopedVector<autofill::PasswordForm> logins;
-  FillBlacklistLogins(&logins);
-  logins.swap(*request->result());
-  ForwardLoginsResult(request);
+  FillBlacklistLogins(request->result());
+  ForwardLoginsResult(request.Pass());
 }
 
 bool PasswordStoreDefault::FillAutofillableLogins(