PasswordStore refactoring.

chrome/browser/password_manager/password_store_x_unittest.cc

Index: chrome/browser/password_manager/password_store_x_unittest.cc
diff --git a/chrome/browser/password_manager/password_store_x_unittest.cc b/chrome/browser/password_manager/password_store_x_unittest.cc
index 4a67b5d05aafef7320ce33e64cb28bc2c855208c..ac1742630ee6ce67ac5fee1077ebe47fbf210580 100644
--- a/chrome/browser/password_manager/password_store_x_unittest.cc
+++ b/chrome/browser/password_manager/password_store_x_unittest.cc
@@ -68,8 +68,10 @@ class FailingBackend : public PasswordStoreX::NativeBackend {
   }
   virtual bool RemoveLogin(const PasswordForm& form) OVERRIDE { return false; }
 
-  virtual bool RemoveLoginsCreatedBetween(base::Time delete_begin,
-                                          base::Time delete_end) OVERRIDE {
+  virtual bool RemoveLoginsCreatedBetween(
+      base::Time delete_begin,
+      base::Time delete_end,
+      password_manager::PasswordStoreChangeList* changes) OVERRIDE {
     return false;
   }
 
@@ -85,12 +87,6 @@ class FailingBackend : public PasswordStoreX::NativeBackend {
     return false;
   }
 
-  virtual bool GetLoginsCreatedBetween(base::Time get_begin,
-                                       base::Time get_end,
-                                       PasswordFormList* forms) OVERRIDE {
-    return false;
-  }
-
   virtual bool GetAutofillableLogins(PasswordFormList* forms) OVERRIDE {
     return false;
   }
@@ -127,8 +123,10 @@ class MockBackend : public PasswordStoreX::NativeBackend {
     return true;
   }
 
-  virtual bool RemoveLoginsCreatedBetween(base::Time delete_begin,
-                                          base::Time delete_end) OVERRIDE {
+  virtual bool RemoveLoginsCreatedBetween(
+      base::Time delete_begin,
+      base::Time delete_end,
+      password_manager::PasswordStoreChangeList* changes) OVERRIDE {
     for (size_t i = 0; i < all_forms_.size(); ++i) {
       if (delete_begin <= all_forms_[i].date_created &&
           (delete_end.is_null() || all_forms_[i].date_created < delete_end))
@@ -161,16 +159,6 @@ class MockBackend : public PasswordStoreX::NativeBackend {
     return true;
   }
 
-  virtual bool GetLoginsCreatedBetween(base::Time get_begin,
-                                       base::Time get_end,
-                                       PasswordFormList* forms) OVERRIDE {
-    for (size_t i = 0; i < all_forms_.size(); ++i)
-      if (get_begin <= all_forms_[i].date_created &&
-          (get_end.is_null() || all_forms_[i].date_created < get_end))
-        forms->push_back(new PasswordForm(all_forms_[i]));
-    return true;
-  }
-
   virtual bool GetAutofillableLogins(PasswordFormList* forms) OVERRIDE {
     for (size_t i = 0; i < all_forms_.size(); ++i)
       if (!all_forms_[i].blacklisted_by_user)