PasswordStoreMac::RemoveLoginsCreatedBetween() shouldn't affect other profiles.

components/password_manager/core/browser/login_database.h

Index: components/password_manager/core/browser/login_database.h
diff --git a/components/password_manager/core/browser/login_database.h b/components/password_manager/core/browser/login_database.h
index ac6bd0be6349d353d8421c7854e40c7a197d4b7f..e58b524d429895ae84bac05512c02195087b7d12 100644
--- a/components/password_manager/core/browser/login_database.h
+++ b/components/password_manager/core/browser/login_database.h
@@ -62,13 +62,13 @@ class LoginDatabase {
 
   // Loads a list of matching password forms into the specified vector |forms|.
   // The list will contain all possibly relevant entries to the observed |form|,
-  // including blacklisted matches.
+  // including blacklisted matches. The caller ownes |forms| after the call.

[vabr (Chromium), 2014/07/02 14:10:35]

typo: ownes->owns (also below)

[vasilii, 2014/07/02 15:28:32]

Done.

   bool GetLogins(const autofill::PasswordForm& form,
                  std::vector<autofill::PasswordForm*>* forms) const;
 
   // Loads all logins created from |begin| onwards (inclusive) and before |end|.
   // You may use a null Time value to do an unbounded search in either
-  // direction.
+  // direction. The caller ownes |forms| after the call.
   bool GetLoginsCreatedBetween(
       base::Time begin,
       base::Time end,
@@ -76,18 +76,19 @@ class LoginDatabase {
 
   // Loads all logins synced from |begin| onwards (inclusive) and before |end|.
   // You may use a null Time value to do an unbounded search in either
-  // direction.
+  // direction. The caller ownes |forms| after the call.
   bool GetLoginsSyncedBetween(
       base::Time begin,
       base::Time end,
       std::vector<autofill::PasswordForm*>* forms) const;
 
   // Loads the complete list of autofillable password forms (i.e., not blacklist
-  // entries) into |forms|.
+  // entries) into |forms|. The caller ownes |forms| after the call.
   bool GetAutofillableLogins(
       std::vector<autofill::PasswordForm*>* forms) const;
 
-  // Loads the complete list of blacklist forms into |forms|.
+  // Loads the complete list of blacklist forms into |forms|. The caller ownes
+  // |forms| after the call.
   bool GetBlacklistLogins(
       std::vector<autofill::PasswordForm*>* forms) const;