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

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 81c4ca0a631527f37b6dff32f044c74e95136cab..88f6e9e45fe47f83a99df12fad70d62a5dd9e340 100644
--- a/components/password_manager/core/browser/password_store_default.cc
+++ b/components/password_manager/core/browser/password_store_default.cc
@@ -122,16 +122,16 @@ ScopedVector<autofill::PasswordForm> PasswordStoreDefault::FillMatchingLogins(
 
 void PasswordStoreDefault::GetAutofillableLoginsImpl(
     scoped_ptr<GetLoginsRequest> request) {
-  DCHECK(request->result()->empty());
-  FillAutofillableLogins(request->result());
-  ForwardLoginsResult(request.Pass());
+  ScopedVector<autofill::PasswordForm> logins;
+  FillAutofillableLogins(&logins);
+  request->NotifyConsumerWithResults(logins.Pass());
 }
 
 void PasswordStoreDefault::GetBlacklistLoginsImpl(
     scoped_ptr<GetLoginsRequest> request) {
-  DCHECK(request->result()->empty());
-  FillBlacklistLogins(request->result());
-  ForwardLoginsResult(request.Pass());
+  ScopedVector<autofill::PasswordForm> logins;
+  FillBlacklistLogins(&logins);
+  request->NotifyConsumerWithResults(logins.Pass());
 }
 
 bool PasswordStoreDefault::FillAutofillableLogins(