PasswordStore: Use ScopedVector to express ownership of forms

chrome/browser/password_manager/native_backend_libsecret.cc

 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/password_manager/native_backend_libsecret.h"
 
 #include <dlfcn.h>
 #include <list>
 
 #include "base/basictypes.h"
@@ skipping 246 matching lines @@
 }
 
 password_manager::PasswordStoreChangeList NativeBackendLibsecret::AddLogin(
     const PasswordForm& form) {
   // Based on LoginDatabase::AddLogin(), we search for an existing match based
   // on origin_url, username_element, username_value, password_element, submit
   // element, and signon_realm first, remove that, and then add the new entry.
   // We'd add the new one first, and then delete the original, but then the
   // delete might actually delete the newly-added entry!
   ScopedVector<autofill::PasswordForm> forms;
-  AddUpdateLoginSearch(form, &forms.get(), SEARCH_USE_SUBMIT);
+  AddUpdateLoginSearch(form, SEARCH_USE_SUBMIT, &forms);
   password_manager::PasswordStoreChangeList changes;
   if (forms.size() > 0) {
     if (forms.size() > 1) {
       VLOG(1) << "Adding login when there are " << forms.size()
               << " matching logins already! Will replace only the first.";
     }
 
     if (RemoveLogin(*forms[0])) {
       changes.push_back(password_manager::PasswordStoreChange(
           password_manager::PasswordStoreChange::REMOVE, *forms[0]));
@@ skipping 12 matching lines @@
   // Based on LoginDatabase::UpdateLogin(), we search for forms to update by
   // origin_url, username_element, username_value, password_element, and
   // signon_realm. We then compare the result to the updated form. If they
   // differ in any of the mutable fields, then we remove the original, and
   // then add the new entry. We'd add the new one first, and then delete the
   // original, but then the delete might actually delete the newly-added entry!
   DCHECK(changes);
   changes->clear();
 
   ScopedVector<autofill::PasswordForm> forms;
-  AddUpdateLoginSearch(form, &forms.get(), SEARCH_IGNORE_SUBMIT);
+  AddUpdateLoginSearch(form, SEARCH_IGNORE_SUBMIT, &forms);
 
   bool removed = false;
   for (size_t i = 0; i < forms.size(); ++i) {
     if (*forms[i] != form) {
       RemoveLogin(*forms[i]);
       removed = true;
     }
   }
   if (!removed)
     return true;
@@ skipping 34 matching lines @@
                              changes);
 }
 
 bool NativeBackendLibsecret::RemoveLoginsSyncedBetween(
     base::Time delete_begin,
     base::Time delete_end,
     password_manager::PasswordStoreChangeList* changes) {
   return RemoveLoginsBetween(delete_begin, delete_end, SYNC_TIMESTAMP, changes);
 }
 
-bool NativeBackendLibsecret::GetLogins(const PasswordForm& form,
-                                       PasswordFormList* forms) {
-  return GetLoginsList(forms, &form, ALL_LOGINS);
+bool NativeBackendLibsecret::GetLogins(
+    const PasswordForm& form,
+    ScopedVector<autofill::PasswordForm>* forms) {
+  return GetLoginsList(&form, ALL_LOGINS, forms);
 }
 
 void NativeBackendLibsecret::AddUpdateLoginSearch(
     const autofill::PasswordForm& lookup_form,
-    PasswordFormList* forms,
-    AddUpdateLoginSearchOptions options) {
+    AddUpdateLoginSearchOptions options,
+    ScopedVector<autofill::PasswordForm>* forms) {
   LibsecretAttributesBuilder attrs;
   attrs.Append("origin_url", lookup_form.origin.spec());
   attrs.Append("username_element", UTF16ToUTF8(lookup_form.username_element));
   attrs.Append("username_value", UTF16ToUTF8(lookup_form.username_value));
   attrs.Append("password_element", UTF16ToUTF8(lookup_form.password_element));
   if (options == SEARCH_USE_SUBMIT)
     attrs.Append("submit_element", UTF16ToUTF8(lookup_form.submit_element));
   attrs.Append("signon_realm", lookup_form.signon_realm);
   attrs.Append("application", app_string_);
 
@@ skipping 45 matching lines @@
       "application", app_string_.c_str(), nullptr);
 
   if (error) {
     VLOG(1) << "Libsecret add raw login failed: " << error->message;
     g_error_free(error);
     return false;
   }
   return true;
 }
 
-bool NativeBackendLibsecret::GetAutofillableLogins(PasswordFormList* forms) {
-  return GetLoginsList(forms, nullptr, AUTOFILLABLE_LOGINS);
+bool NativeBackendLibsecret::GetAutofillableLogins(
+    ScopedVector<autofill::PasswordForm>* forms) {
+  return GetLoginsList(nullptr, AUTOFILLABLE_LOGINS, forms);
 }
 
-bool NativeBackendLibsecret::GetBlacklistLogins(PasswordFormList* forms) {
-  return GetLoginsList(forms, nullptr, BLACKLISTED_LOGINS);
+bool NativeBackendLibsecret::GetBlacklistLogins(
+    ScopedVector<autofill::PasswordForm>* forms) {
+  return GetLoginsList(nullptr, BLACKLISTED_LOGINS, forms);
 }
 
-bool NativeBackendLibsecret::GetLoginsList(PasswordFormList* forms,
-                                           const PasswordForm* lookup_form,
-                                           GetLoginsListOptions options) {
+bool NativeBackendLibsecret::GetLoginsList(
+    const PasswordForm* lookup_form,
+    GetLoginsListOptions options,
+    ScopedVector<autofill::PasswordForm>* forms) {
   LibsecretAttributesBuilder attrs;
   attrs.Append("application", app_string_);
   if (options != ALL_LOGINS)
     attrs.Append("blacklisted_by_user", options == BLACKLISTED_LOGINS);
   if (lookup_form &&
       !password_manager::ShouldPSLDomainMatchingApply(
           password_manager::GetRegistryControlledDomain(
               GURL(lookup_form->signon_realm))))
     attrs.Append("signon_realm", lookup_form->signon_realm);
 
   GError* error = nullptr;
   GList* found = secret_service_search_sync(nullptr,  // default secret service
                                             &kLibsecretSchema, attrs.Get(),
                                             SECRET_SEARCH_ALL,
                                             nullptr,  // no cancellable ojbect
                                             &error);
   if (error) {
     VLOG(1) << "Unable to get logins " << error->message;
     g_error_free(error);
     if (found)
       g_list_free(found);
     return false;
   }
 
   return ConvertFormList(found, lookup_form, forms);
 }
 
-bool NativeBackendLibsecret::GetAllLogins(PasswordFormList* forms) {
-  return GetLoginsList(forms, nullptr, ALL_LOGINS);
+bool NativeBackendLibsecret::GetAllLogins(
+    ScopedVector<autofill::PasswordForm>* forms) {
+  return GetLoginsList(nullptr, ALL_LOGINS, forms);
 }
 
 bool NativeBackendLibsecret::GetLoginsBetween(
     base::Time get_begin,
     base::Time get_end,
     TimestampToCompare date_to_compare,
-    PasswordFormList* forms) {
-  PasswordFormList all_forms;
+    ScopedVector<autofill::PasswordForm>* forms) {
+  ScopedVector<autofill::PasswordForm> all_forms;
   if (!GetAllLogins(&all_forms))
     return false;
 
   base::Time autofill::PasswordForm::*date_member =
       date_to_compare == CREATION_TIMESTAMP
           ? &autofill::PasswordForm::date_created
           : &autofill::PasswordForm::date_synced;
-  for (size_t i = 0; i < all_forms.size(); ++i) {
-    if (get_begin <= all_forms[i]->*date_member &&
-        (get_end.is_null() || all_forms[i]->*date_member < get_end)) {
-      forms->push_back(all_forms[i]);
-    } else {
-      delete all_forms[i];
+  for (auto& saved_form : all_forms) {
+    if (get_begin <= saved_form->*date_member &&
+        (get_end.is_null() || saved_form->*date_member < get_end)) {
+      forms->push_back(saved_form);
+      saved_form = nullptr;
     }
   }
 
   return true;
 }
 
 bool NativeBackendLibsecret::RemoveLoginsBetween(
     base::Time get_begin,
     base::Time get_end,
     TimestampToCompare date_to_compare,
     password_manager::PasswordStoreChangeList* changes) {
   DCHECK(changes);
   changes->clear();
   ScopedVector<autofill::PasswordForm> forms;
-  if (!GetLoginsBetween(get_begin, get_end, date_to_compare, &forms.get()))
+  if (!GetLoginsBetween(get_begin, get_end, date_to_compare, &forms))
     return false;
 
   bool ok = true;
   for (size_t i = 0; i < forms.size(); ++i) {
     if (RemoveLogin(*forms[i])) {
       changes->push_back(password_manager::PasswordStoreChange(
           password_manager::PasswordStoreChange::REMOVE, *forms[i]));
     } else {
       ok = false;
     }
   }
   return ok;
 }
 
 bool NativeBackendLibsecret::ConvertFormList(
     GList* found,
     const PasswordForm* lookup_form,
-    NativeBackendLibsecret::PasswordFormList* forms) {
+    ScopedVector<autofill::PasswordForm>* forms) {
   password_manager::PSLDomainMatchMetric psl_domain_match_metric =
       password_manager::PSL_DOMAIN_MATCH_NONE;
   GError* error = nullptr;
   for (GList* element = g_list_first(found); element != nullptr;
        element = g_list_next(element)) {
     SecretItem* secretItem = static_cast<SecretItem*>(element->data);
     LibsecretLoader::secret_item_load_secret_sync(secretItem, nullptr, &error);
     if (error) {
       VLOG(1) << "Unable to load secret item" << error->message;
       g_error_free(error);
@@ skipping 45 matching lines @@
   return true;
 }
 
 std::string NativeBackendLibsecret::GetProfileSpecificAppString(
     LocalProfileId id) {
   // Originally, the application string was always just "chrome" and used only
   // so that we had *something* to search for since GNOME Keyring won't search
   // for nothing. Now we use it to distinguish passwords for different profiles.
   return base::StringPrintf("%s-%d", kLibsecretAppString, id);
 }