Implement origin-based deletion for passwords in PasswordDefaultMac.

components/password_manager/core/browser/password_store_default_unittest.cc

Index: components/password_manager/core/browser/password_store_default_unittest.cc
diff --git a/components/password_manager/core/browser/password_store_default_unittest.cc b/components/password_manager/core/browser/password_store_default_unittest.cc
index b9e9c5811b6fae0d30e142fade0b195aa514eb75..5052dbbe26fdc2c40b647210e7c24e3dfff3106a 100644
--- a/components/password_manager/core/browser/password_store_default_unittest.cc
+++ b/components/password_manager/core/browser/password_store_default_unittest.cc
@@ -18,6 +18,7 @@
 #include "components/password_manager/core/browser/password_store_change.h"
 #include "components/password_manager/core/browser/password_store_consumer.h"
 #include "components/password_manager/core/browser/password_store_default.h"
+#include "components/password_manager/core/browser/password_store_origin_unittest.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/origin.h"
@@ -43,11 +44,6 @@ class MockPasswordStoreConsumer : public PasswordStoreConsumer {
   }
 };
 
-class MockPasswordStoreObserver : public PasswordStore::Observer {
- public:
-  MOCK_METHOD1(OnLoginsChanged, void(const PasswordStoreChangeList& changes));
-};
-
 // A mock LoginDatabase that simulates a failing Init() method.
 class BadLoginDatabase : public LoginDatabase {
  public:
@@ -79,64 +75,92 @@ PasswordFormData CreateTestPasswordFormData() {
   return data;
 }
 
-PasswordFormData CreateTestPasswordFormDataByOrigin(const char* origin_url) {
-  PasswordFormData data = {PasswordForm::SCHEME_HTML,
-                           origin_url,
-                           origin_url,
-                           origin_url,
-                           L"submit_element",
-                           L"username_element",
-                           L"password_element",
-                           L"username_value",
-                           L"password_value",
-                           true,
-                           false,
-                           base::Time::Now().ToDoubleT()};
-  return data;
-}
-
 }  // anonymous namespace
 
-class PasswordStoreDefaultTest : public testing::Test {
- protected:
-  void SetUp() override {
-    ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
-  }
+class PasswordStoreDefaultTestDelegate {

[vasilii, 2015/11/26 11:03:38]

You can move it to the anonymous namespace

[Timo Reimann, 2015/11/26 14:38:29]

Done.

+ public:
+  PasswordStoreDefaultTestDelegate();
+  explicit PasswordStoreDefaultTestDelegate(scoped_ptr<LoginDatabase> database);
+  ~PasswordStoreDefaultTestDelegate();
 
-  void TearDown() override {
-    base::MessageLoop::current()->RunUntilIdle();
-    ASSERT_TRUE(temp_dir_.Delete());
-  }
+  PasswordStoreDefault* store() { return store_.get(); }
 
-  base::FilePath test_login_db_file_path() const {
-    return temp_dir_.path().Append(FILE_PATH_LITERAL("login_test"));
-  }
+  static void FinishAsyncProcessing();
 
-  scoped_refptr<PasswordStoreDefault> CreateInitializedStore() {
-    return CreateInitializedStore(
-        make_scoped_ptr(new LoginDatabase(test_login_db_file_path())));
-  }
+ private:
+  void SetupTempDir();
+
+  void ClosePasswordStore();
 
   scoped_refptr<PasswordStoreDefault> CreateInitializedStore(
-      scoped_ptr<LoginDatabase> database) {
-    scoped_refptr<PasswordStoreDefault> store(new PasswordStoreDefault(
-        base::ThreadTaskRunnerHandle::Get(),
-        base::ThreadTaskRunnerHandle::Get(), database.Pass()));
-    store->Init(syncer::SyncableService::StartSyncFlare());
+      scoped_ptr<LoginDatabase> database);
 
-    return store;
-  }
+  base::FilePath test_login_db_file_path() const;
 
   base::MessageLoopForUI message_loop_;
   base::ScopedTempDir temp_dir_;
+  scoped_refptr<PasswordStoreDefault> store_;
+
+  DISALLOW_COPY_AND_ASSIGN(PasswordStoreDefaultTestDelegate);
 };
 
+PasswordStoreDefaultTestDelegate::PasswordStoreDefaultTestDelegate() {
+  SetupTempDir();
+  store_ = CreateInitializedStore(
+      make_scoped_ptr(new LoginDatabase(test_login_db_file_path())));
+}
+
+PasswordStoreDefaultTestDelegate::PasswordStoreDefaultTestDelegate(
+    scoped_ptr<LoginDatabase> database) {
+  SetupTempDir();
+  store_ = CreateInitializedStore(database.Pass());
+}
+
+PasswordStoreDefaultTestDelegate::~PasswordStoreDefaultTestDelegate() {
+  ClosePasswordStore();
+}
+
+void PasswordStoreDefaultTestDelegate::SetupTempDir() {
+  ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
+}
+
+void PasswordStoreDefaultTestDelegate::ClosePasswordStore() {
+  store_->ShutdownOnUIThread();
+  base::MessageLoop::current()->RunUntilIdle();
+  ASSERT_TRUE(temp_dir_.Delete());
+}
+
+scoped_refptr<PasswordStoreDefault>
+PasswordStoreDefaultTestDelegate::CreateInitializedStore(
+    scoped_ptr<LoginDatabase> database) {
+  scoped_refptr<PasswordStoreDefault> store(new PasswordStoreDefault(
+      base::ThreadTaskRunnerHandle::Get(), base::ThreadTaskRunnerHandle::Get(),
+      database.Pass()));
+  store->Init(syncer::SyncableService::StartSyncFlare());
+
+  return store;
+}
+
+base::FilePath PasswordStoreDefaultTestDelegate::test_login_db_file_path()
+    const {
+  return temp_dir_.path().Append(FILE_PATH_LITERAL("login_test"));
+}
+
+void PasswordStoreDefaultTestDelegate::FinishAsyncProcessing() {

[vasilii, 2015/11/26 11:03:38]

Declaration order and definition order should match each other.

[Timo Reimann, 2015/11/26 14:38:29]

Updated.

+  base::MessageLoop::current()->RunUntilIdle();
+}
+
+INSTANTIATE_TYPED_TEST_CASE_P(Default,
+                              PasswordStoreOriginTest,
+                              PasswordStoreDefaultTestDelegate);
+
 ACTION(STLDeleteElements0) {
   STLDeleteContainerPointers(arg0.begin(), arg0.end());
 }
 
-TEST_F(PasswordStoreDefaultTest, NonASCIIData) {
-  scoped_refptr<PasswordStoreDefault> store = CreateInitializedStore();
+TEST(PasswordStoreDefaultTest, NonASCIIData) {
+  PasswordStoreDefaultTestDelegate delegate;
+  PasswordStoreDefault* store = delegate.store();
 
   // Some non-ASCII password form data.
   static const PasswordFormData form_data[] = {
@@ -171,12 +195,11 @@ TEST_F(PasswordStoreDefaultTest, NonASCIIData) {
   store->GetAutofillableLogins(&consumer);
 
   base::MessageLoop::current()->RunUntilIdle();
-
-  store->ShutdownOnUIThread();
 }
 
-TEST_F(PasswordStoreDefaultTest, Notifications) {
-  scoped_refptr<PasswordStoreDefault> store = CreateInitializedStore();
+TEST(PasswordStoreDefaultTest, Notifications) {
+  PasswordStoreDefaultTestDelegate delegate;
+  PasswordStoreDefault* store = delegate.store();
 
   scoped_ptr<PasswordForm> form =
       CreatePasswordFormFromDataForTesting(CreateTestPasswordFormData());
@@ -221,16 +244,15 @@ TEST_F(PasswordStoreDefaultTest, Notifications) {
   base::MessageLoop::current()->RunUntilIdle();
 
   store->RemoveObserver(&observer);
-  store->ShutdownOnUIThread();
 }
 
 // Verify that operations on a PasswordStore with a bad database cause no
 // explosions, but fail without side effect, return no data and trigger no
 // notifications.
-TEST_F(PasswordStoreDefaultTest, OperationsOnABadDatabaseSilentlyFail) {
-  scoped_refptr<PasswordStoreDefault> bad_store =
-      CreateInitializedStore(make_scoped_ptr(new BadLoginDatabase));
-
+TEST(PasswordStoreDefaultTest, OperationsOnABadDatabaseSilentlyFail) {
+  PasswordStoreDefaultTestDelegate delegate(
+      make_scoped_ptr<LoginDatabase>(new BadLoginDatabase));

[vasilii, 2015/11/26 11:03:38]

It should work without the type.

[Timo Reimann, 2015/11/26 14:38:29]

Removed.

+  PasswordStoreDefault* bad_store = delegate.store();
   base::MessageLoop::current()->RunUntilIdle();
   ASSERT_EQ(nullptr, bad_store->login_db());
 
@@ -285,85 +307,6 @@ TEST_F(PasswordStoreDefaultTest, OperationsOnABadDatabaseSilentlyFail) {
 
   // Ensure no notifications and no explosions during shutdown either.
   bad_store->RemoveObserver(&mock_observer);
-  bad_store->ShutdownOnUIThread();
-}
-
-TEST_F(PasswordStoreDefaultTest,
-       RemoveLoginsByOriginAndTimeImpl_FittingOriginAndTime) {
-  scoped_refptr<PasswordStoreDefault> store = CreateInitializedStore();
-
-  const char origin_url[] = "http://foo.example.com";
-  scoped_ptr<autofill::PasswordForm> form =
-      CreatePasswordFormFromDataForTesting(
-          CreateTestPasswordFormDataByOrigin(origin_url));
-  store->AddLogin(*form);
-  base::MessageLoop::current()->RunUntilIdle();
-
-  MockPasswordStoreObserver observer;
-  store->AddObserver(&observer);
-  EXPECT_CALL(observer, OnLoginsChanged(ElementsAre(PasswordStoreChange(
-                            PasswordStoreChange::REMOVE, *form))));
-
-  const url::Origin origin((GURL(origin_url)));
-  base::RunLoop run_loop;
-  store->RemoveLoginsByOriginAndTime(origin, base::Time(), base::Time::Max(),
-                                     run_loop.QuitClosure());
-  run_loop.Run();
-
-  store->RemoveObserver(&observer);
-  store->ShutdownOnUIThread();
-}
-
-TEST_F(PasswordStoreDefaultTest,
-       RemoveLoginsByOriginAndTimeImpl_NonMatchingOrigin) {
-  scoped_refptr<PasswordStoreDefault> store = CreateInitializedStore();
-
-  const char origin_url[] = "http://foo.example.com";
-  scoped_ptr<autofill::PasswordForm> form =
-      CreatePasswordFormFromDataForTesting(
-          CreateTestPasswordFormDataByOrigin(origin_url));
-  store->AddLogin(*form);
-  base::MessageLoop::current()->RunUntilIdle();
-
-  MockPasswordStoreObserver observer;
-  store->AddObserver(&observer);
-  EXPECT_CALL(observer, OnLoginsChanged(_)).Times(0);
-
-  const url::Origin other_origin(GURL("http://bar.example.com"));
-  base::RunLoop run_loop;
-  store->RemoveLoginsByOriginAndTime(other_origin, base::Time(),
-                                     base::Time::Max(), run_loop.QuitClosure());
-  run_loop.Run();
-
-  store->RemoveObserver(&observer);
-  store->ShutdownOnUIThread();
-}
-
-TEST_F(PasswordStoreDefaultTest,
-       RemoveLoginsByOriginAndTimeImpl_NotWithinTimeInterval) {
-  scoped_refptr<PasswordStoreDefault> store = CreateInitializedStore();
-
-  const char origin_url[] = "http://foo.example.com";
-  scoped_ptr<autofill::PasswordForm> form =
-      CreatePasswordFormFromDataForTesting(
-          CreateTestPasswordFormDataByOrigin(origin_url));
-  store->AddLogin(*form);
-  base::MessageLoop::current()->RunUntilIdle();
-
-  MockPasswordStoreObserver observer;
-  store->AddObserver(&observer);
-  EXPECT_CALL(observer, OnLoginsChanged(_)).Times(0);
-
-  const url::Origin origin((GURL(origin_url)));
-  base::Time time_after_creation_date =
-      form->date_created + base::TimeDelta::FromDays(1);
-  base::RunLoop run_loop;
-  store->RemoveLoginsByOriginAndTime(origin, time_after_creation_date,
-                                     base::Time::Max(), run_loop.QuitClosure());
-  run_loop.Run();
-
-  store->RemoveObserver(&observer);
-  store->ShutdownOnUIThread();
 }
 
 }  // namespace password_manager