PasswordStore refactoring.

chrome/browser/password_manager/password_store_x.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_PASSWORD_MANAGER_PASSWORD_STORE_X_H_
 #define CHROME_BROWSER_PASSWORD_MANAGER_PASSWORD_STORE_X_H_
 
 #include <vector>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 32 matching lines @@
     virtual password_manager::PasswordStoreChangeList AddLogin(
         const autofill::PasswordForm& form) = 0;
     virtual bool UpdateLogin(
         const autofill::PasswordForm& form,
         password_manager::PasswordStoreChangeList* changes) = 0;
     virtual bool RemoveLogin(const autofill::PasswordForm& form) = 0;
 
     // Removes all logins created/synced from |delete_begin| onwards (inclusive)
     // and before |delete_end|. You may use a null Time value to do an unbounded
     // delete in either direction.
-    virtual bool RemoveLoginsCreatedBetween(base::Time delete_begin,
-                                            base::Time delete_end) = 0;
+    virtual bool RemoveLoginsCreatedBetween(
+        base::Time delete_begin,
+        base::Time delete_end,
+        password_manager::PasswordStoreChangeList* changes) = 0;
     virtual bool RemoveLoginsSyncedBetween(
         base::Time delete_begin,
         base::Time delete_end,
         password_manager::PasswordStoreChangeList* changes) = 0;
 
     virtual bool GetLogins(const autofill::PasswordForm& form,
                            PasswordFormList* forms) = 0;
-    virtual bool GetLoginsCreatedBetween(base::Time get_begin,
-                                         base::Time get_end,
-                                         PasswordFormList* forms) = 0;
     virtual bool GetAutofillableLogins(PasswordFormList* forms) = 0;
     virtual bool GetBlacklistLogins(PasswordFormList* forms) = 0;
   };
 
   // Takes ownership of |login_db| and |backend|. |backend| may be NULL in which
   // case this PasswordStoreX will act the same as PasswordStoreDefault.
   PasswordStoreX(scoped_refptr<base::SingleThreadTaskRunner> main_thread_runner,
                  scoped_refptr<base::SingleThreadTaskRunner> db_thread_runner,
                  password_manager::LoginDatabase* login_db,
                  NativeBackend* backend);
 
  private:
   friend class PasswordStoreXTest;
 
   virtual ~PasswordStoreX();
 
   // Implements PasswordStore interface.
   virtual password_manager::PasswordStoreChangeList AddLoginImpl(
       const autofill::PasswordForm& form) OVERRIDE;
   virtual password_manager::PasswordStoreChangeList UpdateLoginImpl(
       const autofill::PasswordForm& form) OVERRIDE;
   virtual password_manager::PasswordStoreChangeList RemoveLoginImpl(
       const autofill::PasswordForm& form) OVERRIDE;
   virtual password_manager::PasswordStoreChangeList
-      RemoveLoginsCreatedBetweenImpl(const base::Time& delete_begin,
-                                     const base::Time& delete_end) OVERRIDE;
+      RemoveLoginsCreatedBetweenImpl(base::Time delete_begin,
+                                     base::Time delete_end) OVERRIDE;
   virtual password_manager::PasswordStoreChangeList
       RemoveLoginsSyncedBetweenImpl(base::Time delete_begin,
                                     base::Time delete_end) OVERRIDE;
   virtual void GetLoginsImpl(
       const autofill::PasswordForm& form,
       AuthorizationPromptPolicy prompt_policy,
       const ConsumerCallbackRunner& callback_runner) OVERRIDE;
   virtual void GetAutofillableLoginsImpl(GetLoginsRequest* request) OVERRIDE;
   virtual void GetBlacklistLoginsImpl(GetLoginsRequest* request) OVERRIDE;
   virtual bool FillAutofillableLogins(
@@ skipping 27 matching lines @@
   // Whether we should allow falling back to the default store. If there is
   // nothing to migrate, then the first attempt to use the native store will
   // be the first time we try to use it and we should allow falling back. If
   // we have migrated successfully, then we do not allow falling back.
   bool allow_fallback_;
 
   DISALLOW_COPY_AND_ASSIGN(PasswordStoreX);
 };
 
 #endif  // CHROME_BROWSER_PASSWORD_MANAGER_PASSWORD_STORE_X_H_