GetLoginsRequest: 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 678f7c7520e0faa1a5990e86165ffa3b2afe4c9e..03994808616609ebf8c1f58b03039f32ea3d728e 100644
--- a/components/password_manager/core/browser/test_password_store.cc
+++ b/components/password_manager/core/browser/test_password_store.cc
@@ -46,12 +46,12 @@ bool TestPasswordStore::FormsAreEquivalent(const autofill::PasswordForm& lhs,
 }
 
 void TestPasswordStore::GetAutofillableLoginsImpl(
-    PasswordStore::GetLoginsRequest* request) {
+    scoped_ptr<GetLoginsRequest> request) {
   for (auto& forms_for_realm : stored_passwords_) {

[vasilii, 2015/02/09 10:08:35]

const auto&

[vabr (Chromium), 2015/02/09 11:36:06]

Done.

     for (const autofill::PasswordForm& form : forms_for_realm.second)
       request->result()->push_back(new autofill::PasswordForm(form));
   }
-  ForwardLoginsResult(request);
+  ForwardLoginsResult(request.Pass());
 }
 
 PasswordStoreChangeList TestPasswordStore::AddLoginImpl(
@@ -107,6 +107,10 @@ void TestPasswordStore::GetLoginsImpl(
   runner.Run(matched_forms.Pass());
 }
 
+void TestPasswordStore::ReportMetricsImpl(const std::string& sync_username,
+                                          bool custom_passphrase_sync_enabled) {
+}
+
 PasswordStoreChangeList TestPasswordStore::RemoveLoginsCreatedBetweenImpl(
     base::Time begin,
     base::Time end) {
@@ -121,6 +125,10 @@ PasswordStoreChangeList TestPasswordStore::RemoveLoginsSyncedBetweenImpl(
   return changes;
 }
 
+void TestPasswordStore::GetBlacklistLoginsImpl(
+    scoped_ptr<GetLoginsRequest> request) {
+}
+
 bool TestPasswordStore::FillAutofillableLogins(
     ScopedVector<autofill::PasswordForm>* forms) {
   return true;