| OLD | NEW |
| 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "base/basictypes.h" | 5 #include "base/basictypes.h" |
| 6 #include "base/bind.h" | 6 #include "base/bind.h" |
| 7 #include "base/file_util.h" | 7 #include "base/file_util.h" |
| 8 #include "base/platform_file.h" | 8 #include "base/platform_file.h" |
| 9 #include "base/scoped_temp_dir.h" | 9 #include "base/scoped_temp_dir.h" |
| 10 #include "base/stl_util.h" | 10 #include "base/stl_util.h" |
| 11 #include "base/string_util.h" | 11 #include "base/string_util.h" |
| 12 #include "base/stringprintf.h" | 12 #include "base/stringprintf.h" |
| 13 #include "base/synchronization/waitable_event.h" | 13 #include "base/synchronization/waitable_event.h" |
| 14 #include "base/test/signaling_task.h" |
| 14 #include "base/time.h" | 15 #include "base/time.h" |
| 15 #include "base/utf_string_conversions.h" | 16 #include "base/utf_string_conversions.h" |
| 16 #include "chrome/browser/password_manager/password_form_data.h" | 17 #include "chrome/browser/password_manager/password_form_data.h" |
| 17 #include "chrome/browser/password_manager/password_store_change.h" | 18 #include "chrome/browser/password_manager/password_store_change.h" |
| 18 #include "chrome/browser/password_manager/password_store_consumer.h" | 19 #include "chrome/browser/password_manager/password_store_consumer.h" |
| 19 #include "chrome/browser/password_manager/password_store_x.h" | 20 #include "chrome/browser/password_manager/password_store_x.h" |
| 20 #include "chrome/browser/prefs/pref_service.h" | 21 #include "chrome/browser/prefs/pref_service.h" |
| 21 #include "chrome/browser/webdata/web_data_service.h" | 22 #include "chrome/browser/webdata/web_data_service.h" |
| 22 #include "chrome/common/chrome_notification_types.h" | 23 #include "chrome/common/chrome_notification_types.h" |
| 23 #include "chrome/common/pref_names.h" | 24 #include "chrome/common/pref_names.h" |
| 24 #include "chrome/test/base/signaling_task.h" | |
| 25 #include "chrome/test/base/testing_browser_process.h" | 25 #include "chrome/test/base/testing_browser_process.h" |
| 26 #include "chrome/test/base/testing_profile.h" | 26 #include "chrome/test/base/testing_profile.h" |
| 27 #include "content/public/browser/notification_details.h" | 27 #include "content/public/browser/notification_details.h" |
| 28 #include "content/public/browser/notification_registrar.h" | 28 #include "content/public/browser/notification_registrar.h" |
| 29 #include "content/public/browser/notification_source.h" | 29 #include "content/public/browser/notification_source.h" |
| 30 #include "content/test/notification_observer_mock.h" | 30 #include "content/test/notification_observer_mock.h" |
| 31 #include "content/test/test_browser_thread.h" | 31 #include "content/test/test_browser_thread.h" |
| 32 #include "testing/gmock/include/gmock/gmock.h" | 32 #include "testing/gmock/include/gmock/gmock.h" |
| 33 #include "testing/gtest/include/gtest/gtest.h" | 33 #include "testing/gtest/include/gtest/gtest.h" |
| 34 | 34 |
| (...skipping 317 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 352 } | 352 } |
| 353 for (VectorOfForms::iterator it = expected_blacklisted.begin(); | 353 for (VectorOfForms::iterator it = expected_blacklisted.begin(); |
| 354 it != expected_blacklisted.end(); ++it) { | 354 it != expected_blacklisted.end(); ++it) { |
| 355 wds_->AddLogin(**it); | 355 wds_->AddLogin(**it); |
| 356 } | 356 } |
| 357 | 357 |
| 358 // The WDS schedules tasks to run on the DB thread so we schedule yet another | 358 // The WDS schedules tasks to run on the DB thread so we schedule yet another |
| 359 // task to notify us that it's safe to carry on with the test. | 359 // task to notify us that it's safe to carry on with the test. |
| 360 WaitableEvent done(false, false); | 360 WaitableEvent done(false, false); |
| 361 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 361 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 362 new SignalingTask(&done)); | 362 new base::SignalingTask(&done)); |
| 363 done.Wait(); | 363 done.Wait(); |
| 364 | 364 |
| 365 // Initializing the PasswordStore should trigger a migration. | 365 // Initializing the PasswordStore should trigger a migration. |
| 366 scoped_refptr<PasswordStoreX> store( | 366 scoped_refptr<PasswordStoreX> store( |
| 367 new PasswordStoreX(login_db_.release(), | 367 new PasswordStoreX(login_db_.release(), |
| 368 profile_.get(), | 368 profile_.get(), |
| 369 wds_.get(), | 369 wds_.get(), |
| 370 GetBackend())); | 370 GetBackend())); |
| 371 store->Init(); | 371 store->Init(); |
| 372 | 372 |
| 373 // Check that the migration preference has not been initialized. | 373 // Check that the migration preference has not been initialized. |
| 374 ASSERT_TRUE(NULL == profile_->GetPrefs()->FindPreference( | 374 ASSERT_TRUE(NULL == profile_->GetPrefs()->FindPreference( |
| 375 prefs::kLoginDatabaseMigrated)); | 375 prefs::kLoginDatabaseMigrated)); |
| 376 | 376 |
| 377 // Again, the WDS schedules tasks to run on the DB thread, so schedule a task | 377 // Again, the WDS schedules tasks to run on the DB thread, so schedule a task |
| 378 // to signal us when it is safe to continue. | 378 // to signal us when it is safe to continue. |
| 379 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 379 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 380 new SignalingTask(&done)); | 380 new base::SignalingTask(&done)); |
| 381 done.Wait(); | 381 done.Wait(); |
| 382 | 382 |
| 383 // Let the WDS callbacks proceed so the logins can be migrated. | 383 // Let the WDS callbacks proceed so the logins can be migrated. |
| 384 MessageLoop::current()->RunAllPending(); | 384 MessageLoop::current()->RunAllPending(); |
| 385 | 385 |
| 386 MockPasswordStoreConsumer consumer; | 386 MockPasswordStoreConsumer consumer; |
| 387 | 387 |
| 388 // Make sure we quit the MessageLoop even if the test fails. | 388 // Make sure we quit the MessageLoop even if the test fails. |
| 389 ON_CALL(consumer, OnPasswordStoreRequestDone(_, _)) | 389 ON_CALL(consumer, OnPasswordStoreRequestDone(_, _)) |
| 390 .WillByDefault(QuitUIMessageLoop()); | 390 .WillByDefault(QuitUIMessageLoop()); |
| (...skipping 24 matching lines...) Expand all Loading... |
| 415 MockWebDataServiceConsumer wds_consumer; | 415 MockWebDataServiceConsumer wds_consumer; |
| 416 | 416 |
| 417 // No autofillable logins should be left in the WDS. | 417 // No autofillable logins should be left in the WDS. |
| 418 EXPECT_CALL(wds_consumer, | 418 EXPECT_CALL(wds_consumer, |
| 419 OnWebDataServiceRequestDone(_, EmptyWDResult())); | 419 OnWebDataServiceRequestDone(_, EmptyWDResult())); |
| 420 | 420 |
| 421 wds_->GetAutofillableLogins(&wds_consumer); | 421 wds_->GetAutofillableLogins(&wds_consumer); |
| 422 | 422 |
| 423 // Wait for the WDS methods to execute on the DB thread. | 423 // Wait for the WDS methods to execute on the DB thread. |
| 424 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 424 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 425 new SignalingTask(&done)); | 425 new base::SignalingTask(&done)); |
| 426 done.Wait(); | 426 done.Wait(); |
| 427 | 427 |
| 428 // Handle the callback from the WDS. | 428 // Handle the callback from the WDS. |
| 429 MessageLoop::current()->RunAllPending(); | 429 MessageLoop::current()->RunAllPending(); |
| 430 | 430 |
| 431 // Likewise, no blacklisted logins should be left in the WDS. | 431 // Likewise, no blacklisted logins should be left in the WDS. |
| 432 EXPECT_CALL(wds_consumer, | 432 EXPECT_CALL(wds_consumer, |
| 433 OnWebDataServiceRequestDone(_, EmptyWDResult())); | 433 OnWebDataServiceRequestDone(_, EmptyWDResult())); |
| 434 | 434 |
| 435 wds_->GetBlacklistLogins(&wds_consumer); | 435 wds_->GetBlacklistLogins(&wds_consumer); |
| 436 | 436 |
| 437 // Wait for the WDS methods to execute on the DB thread. | 437 // Wait for the WDS methods to execute on the DB thread. |
| 438 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 438 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 439 new SignalingTask(&done)); | 439 new base::SignalingTask(&done)); |
| 440 done.Wait(); | 440 done.Wait(); |
| 441 | 441 |
| 442 // Handle the callback from the WDS. | 442 // Handle the callback from the WDS. |
| 443 MessageLoop::current()->RunAllPending(); | 443 MessageLoop::current()->RunAllPending(); |
| 444 | 444 |
| 445 STLDeleteElements(&expected_autofillable); | 445 STLDeleteElements(&expected_autofillable); |
| 446 STLDeleteElements(&expected_blacklisted); | 446 STLDeleteElements(&expected_blacklisted); |
| 447 | 447 |
| 448 // Public in PasswordStore, protected in PasswordStoreX. | 448 // Public in PasswordStore, protected in PasswordStoreX. |
| 449 static_cast<PasswordStore*>(store)->Shutdown(); | 449 static_cast<PasswordStore*>(store)->Shutdown(); |
| (...skipping 22 matching lines...) Expand all Loading... |
| 472 // Populate the WDS with logins that should be migrated. | 472 // Populate the WDS with logins that should be migrated. |
| 473 for (VectorOfForms::iterator it = unexpected_autofillable.begin(); | 473 for (VectorOfForms::iterator it = unexpected_autofillable.begin(); |
| 474 it != unexpected_autofillable.end(); ++it) { | 474 it != unexpected_autofillable.end(); ++it) { |
| 475 wds_->AddLogin(**it); | 475 wds_->AddLogin(**it); |
| 476 } | 476 } |
| 477 | 477 |
| 478 // The WDS schedules tasks to run on the DB thread so we schedule yet another | 478 // The WDS schedules tasks to run on the DB thread so we schedule yet another |
| 479 // task to notify us that it's safe to carry on with the test. | 479 // task to notify us that it's safe to carry on with the test. |
| 480 WaitableEvent done(false, false); | 480 WaitableEvent done(false, false); |
| 481 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 481 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 482 new SignalingTask(&done)); | 482 new base::SignalingTask(&done)); |
| 483 done.Wait(); | 483 done.Wait(); |
| 484 | 484 |
| 485 // Pretend that the migration has already taken place. | 485 // Pretend that the migration has already taken place. |
| 486 profile_->GetPrefs()->RegisterBooleanPref(prefs::kLoginDatabaseMigrated, | 486 profile_->GetPrefs()->RegisterBooleanPref(prefs::kLoginDatabaseMigrated, |
| 487 true, | 487 true, |
| 488 PrefService::UNSYNCABLE_PREF); | 488 PrefService::UNSYNCABLE_PREF); |
| 489 | 489 |
| 490 // Initializing the PasswordStore shouldn't trigger a migration. | 490 // Initializing the PasswordStore shouldn't trigger a migration. |
| 491 scoped_refptr<PasswordStoreX> store( | 491 scoped_refptr<PasswordStoreX> store( |
| 492 new PasswordStoreX(login_db_.release(), | 492 new PasswordStoreX(login_db_.release(), |
| (...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 557 Pointee(ElementsAreArray( | 557 Pointee(ElementsAreArray( |
| 558 expected_add_changes))))); | 558 expected_add_changes))))); |
| 559 | 559 |
| 560 // Adding a login should trigger a notification. | 560 // Adding a login should trigger a notification. |
| 561 store->AddLogin(*form); | 561 store->AddLogin(*form); |
| 562 | 562 |
| 563 // The PasswordStore schedules tasks to run on the DB thread so we schedule | 563 // The PasswordStore schedules tasks to run on the DB thread so we schedule |
| 564 // yet another task to notify us that it's safe to carry on with the test. | 564 // yet another task to notify us that it's safe to carry on with the test. |
| 565 WaitableEvent done(false, false); | 565 WaitableEvent done(false, false); |
| 566 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 566 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 567 new SignalingTask(&done)); | 567 new base::SignalingTask(&done)); |
| 568 done.Wait(); | 568 done.Wait(); |
| 569 | 569 |
| 570 // Change the password. | 570 // Change the password. |
| 571 form->password_value = WideToUTF16(L"a different password"); | 571 form->password_value = WideToUTF16(L"a different password"); |
| 572 | 572 |
| 573 const PasswordStoreChange expected_update_changes[] = { | 573 const PasswordStoreChange expected_update_changes[] = { |
| 574 PasswordStoreChange(PasswordStoreChange::UPDATE, *form), | 574 PasswordStoreChange(PasswordStoreChange::UPDATE, *form), |
| 575 }; | 575 }; |
| 576 | 576 |
| 577 EXPECT_CALL(helper->observer(), | 577 EXPECT_CALL(helper->observer(), |
| 578 Observe(int(chrome::NOTIFICATION_LOGINS_CHANGED), | 578 Observe(int(chrome::NOTIFICATION_LOGINS_CHANGED), |
| 579 content::Source<PasswordStore>(store), | 579 content::Source<PasswordStore>(store), |
| 580 Property(&content::Details<const PasswordStoreChangeList>::ptr, | 580 Property(&content::Details<const PasswordStoreChangeList>::ptr, |
| 581 Pointee(ElementsAreArray( | 581 Pointee(ElementsAreArray( |
| 582 expected_update_changes))))); | 582 expected_update_changes))))); |
| 583 | 583 |
| 584 // Updating the login with the new password should trigger a notification. | 584 // Updating the login with the new password should trigger a notification. |
| 585 store->UpdateLogin(*form); | 585 store->UpdateLogin(*form); |
| 586 | 586 |
| 587 // Wait for PasswordStore to send the notification. | 587 // Wait for PasswordStore to send the notification. |
| 588 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 588 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 589 new SignalingTask(&done)); | 589 new base::SignalingTask(&done)); |
| 590 done.Wait(); | 590 done.Wait(); |
| 591 | 591 |
| 592 const PasswordStoreChange expected_delete_changes[] = { | 592 const PasswordStoreChange expected_delete_changes[] = { |
| 593 PasswordStoreChange(PasswordStoreChange::REMOVE, *form), | 593 PasswordStoreChange(PasswordStoreChange::REMOVE, *form), |
| 594 }; | 594 }; |
| 595 | 595 |
| 596 EXPECT_CALL(helper->observer(), | 596 EXPECT_CALL(helper->observer(), |
| 597 Observe(int(chrome::NOTIFICATION_LOGINS_CHANGED), | 597 Observe(int(chrome::NOTIFICATION_LOGINS_CHANGED), |
| 598 content::Source<PasswordStore>(store), | 598 content::Source<PasswordStore>(store), |
| 599 Property(&content::Details<const PasswordStoreChangeList>::ptr, | 599 Property(&content::Details<const PasswordStoreChangeList>::ptr, |
| 600 Pointee(ElementsAreArray( | 600 Pointee(ElementsAreArray( |
| 601 expected_delete_changes))))); | 601 expected_delete_changes))))); |
| 602 | 602 |
| 603 // Deleting the login should trigger a notification. | 603 // Deleting the login should trigger a notification. |
| 604 store->RemoveLogin(*form); | 604 store->RemoveLogin(*form); |
| 605 | 605 |
| 606 // Wait for PasswordStore to send the notification. | 606 // Wait for PasswordStore to send the notification. |
| 607 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 607 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 608 new SignalingTask(&done)); | 608 new base::SignalingTask(&done)); |
| 609 done.Wait(); | 609 done.Wait(); |
| 610 | 610 |
| 611 // Public in PasswordStore, protected in PasswordStoreX. | 611 // Public in PasswordStore, protected in PasswordStoreX. |
| 612 static_cast<PasswordStore*>(store)->Shutdown(); | 612 static_cast<PasswordStore*>(store)->Shutdown(); |
| 613 } | 613 } |
| 614 | 614 |
| 615 TEST_P(PasswordStoreXTest, NativeMigration) { | 615 TEST_P(PasswordStoreXTest, NativeMigration) { |
| 616 VectorOfForms expected_autofillable; | 616 VectorOfForms expected_autofillable; |
| 617 InitExpectedForms(true, 50, &expected_autofillable); | 617 InitExpectedForms(true, 50, &expected_autofillable); |
| 618 | 618 |
| (...skipping 21 matching lines...) Expand all Loading... |
| 640 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 640 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 641 base::IgnoreReturn<bool>(base::Bind( | 641 base::IgnoreReturn<bool>(base::Bind( |
| 642 &LoginDatabase::AddLogin, | 642 &LoginDatabase::AddLogin, |
| 643 base::Unretained(login_db), **it))); | 643 base::Unretained(login_db), **it))); |
| 644 } | 644 } |
| 645 | 645 |
| 646 // Schedule another task on the DB thread to notify us that it's safe to | 646 // Schedule another task on the DB thread to notify us that it's safe to |
| 647 // carry on with the test. | 647 // carry on with the test. |
| 648 WaitableEvent done(false, false); | 648 WaitableEvent done(false, false); |
| 649 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 649 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 650 new SignalingTask(&done)); | 650 new base::SignalingTask(&done)); |
| 651 done.Wait(); | 651 done.Wait(); |
| 652 | 652 |
| 653 // Get the new size of the login DB file. We expect it to be larger. | 653 // Get the new size of the login DB file. We expect it to be larger. |
| 654 base::PlatformFileInfo db_file_full_info; | 654 base::PlatformFileInfo db_file_full_info; |
| 655 ASSERT_TRUE(file_util::GetFileInfo(login_db_file, &db_file_full_info)); | 655 ASSERT_TRUE(file_util::GetFileInfo(login_db_file, &db_file_full_info)); |
| 656 EXPECT_GT(db_file_full_info.size, db_file_start_info.size); | 656 EXPECT_GT(db_file_full_info.size, db_file_start_info.size); |
| 657 | 657 |
| 658 // Pretend that the WDS migration has already taken place. | 658 // Pretend that the WDS migration has already taken place. |
| 659 profile_->GetPrefs()->RegisterBooleanPref(prefs::kLoginDatabaseMigrated, | 659 profile_->GetPrefs()->RegisterBooleanPref(prefs::kLoginDatabaseMigrated, |
| 660 true, | 660 true, |
| (...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 705 OnLoginDatabaseQueryDone( | 705 OnLoginDatabaseQueryDone( |
| 706 ContainsAllPasswordForms(expected_autofillable))) | 706 ContainsAllPasswordForms(expected_autofillable))) |
| 707 .WillOnce(WithArg<0>(STLDeleteElements0())); | 707 .WillOnce(WithArg<0>(STLDeleteElements0())); |
| 708 } | 708 } |
| 709 | 709 |
| 710 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 710 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 711 new LoginDatabaseQueryTask(login_db, true, &ld_return)); | 711 new LoginDatabaseQueryTask(login_db, true, &ld_return)); |
| 712 | 712 |
| 713 // Wait for the login DB methods to execute on the DB thread. | 713 // Wait for the login DB methods to execute on the DB thread. |
| 714 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 714 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 715 new SignalingTask(&done)); | 715 new base::SignalingTask(&done)); |
| 716 done.Wait(); | 716 done.Wait(); |
| 717 | 717 |
| 718 if (GetParam() == WORKING_BACKEND) { | 718 if (GetParam() == WORKING_BACKEND) { |
| 719 // Likewise, no blacklisted logins should be left in the login DB. | 719 // Likewise, no blacklisted logins should be left in the login DB. |
| 720 EXPECT_CALL(ld_return, | 720 EXPECT_CALL(ld_return, |
| 721 OnLoginDatabaseQueryDone(ContainsAllPasswordForms(empty))); | 721 OnLoginDatabaseQueryDone(ContainsAllPasswordForms(empty))); |
| 722 } else { | 722 } else { |
| 723 // The blacklisted logins should still be in the login DB. | 723 // The blacklisted logins should still be in the login DB. |
| 724 EXPECT_CALL(ld_return, | 724 EXPECT_CALL(ld_return, |
| 725 OnLoginDatabaseQueryDone( | 725 OnLoginDatabaseQueryDone( |
| 726 ContainsAllPasswordForms(expected_blacklisted))) | 726 ContainsAllPasswordForms(expected_blacklisted))) |
| 727 .WillOnce(WithArg<0>(STLDeleteElements0())); | 727 .WillOnce(WithArg<0>(STLDeleteElements0())); |
| 728 } | 728 } |
| 729 | 729 |
| 730 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 730 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 731 new LoginDatabaseQueryTask(login_db, false, &ld_return)); | 731 new LoginDatabaseQueryTask(login_db, false, &ld_return)); |
| 732 | 732 |
| 733 // Wait for the login DB methods to execute on the DB thread. | 733 // Wait for the login DB methods to execute on the DB thread. |
| 734 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, | 734 BrowserThread::PostTask(BrowserThread::DB, FROM_HERE, |
| 735 new SignalingTask(&done)); | 735 new base::SignalingTask(&done)); |
| 736 done.Wait(); | 736 done.Wait(); |
| 737 | 737 |
| 738 if (GetParam() == WORKING_BACKEND) { | 738 if (GetParam() == WORKING_BACKEND) { |
| 739 // If the migration succeeded, then not only should there be no logins left | 739 // If the migration succeeded, then not only should there be no logins left |
| 740 // in the login DB, but also the file should have been deleted and then | 740 // in the login DB, but also the file should have been deleted and then |
| 741 // recreated. We approximate checking for this by checking that the file | 741 // recreated. We approximate checking for this by checking that the file |
| 742 // size is equal to the size before we populated it, even though it was | 742 // size is equal to the size before we populated it, even though it was |
| 743 // larger after populating it. | 743 // larger after populating it. |
| 744 base::PlatformFileInfo db_file_end_info; | 744 base::PlatformFileInfo db_file_end_info; |
| 745 ASSERT_TRUE(file_util::GetFileInfo(login_db_file, &db_file_end_info)); | 745 ASSERT_TRUE(file_util::GetFileInfo(login_db_file, &db_file_end_info)); |
| 746 EXPECT_EQ(db_file_start_info.size, db_file_end_info.size); | 746 EXPECT_EQ(db_file_start_info.size, db_file_end_info.size); |
| 747 } | 747 } |
| 748 | 748 |
| 749 STLDeleteElements(&expected_autofillable); | 749 STLDeleteElements(&expected_autofillable); |
| 750 STLDeleteElements(&expected_blacklisted); | 750 STLDeleteElements(&expected_blacklisted); |
| 751 | 751 |
| 752 // Public in PasswordStore, protected in PasswordStoreX. | 752 // Public in PasswordStore, protected in PasswordStoreX. |
| 753 static_cast<PasswordStore*>(store)->Shutdown(); | 753 static_cast<PasswordStore*>(store)->Shutdown(); |
| 754 } | 754 } |
| 755 | 755 |
| 756 INSTANTIATE_TEST_CASE_P(NoBackend, | 756 INSTANTIATE_TEST_CASE_P(NoBackend, |
| 757 PasswordStoreXTest, | 757 PasswordStoreXTest, |
| 758 testing::Values(NO_BACKEND)); | 758 testing::Values(NO_BACKEND)); |
| 759 INSTANTIATE_TEST_CASE_P(FailingBackend, | 759 INSTANTIATE_TEST_CASE_P(FailingBackend, |
| 760 PasswordStoreXTest, | 760 PasswordStoreXTest, |
| 761 testing::Values(FAILING_BACKEND)); | 761 testing::Values(FAILING_BACKEND)); |
| 762 INSTANTIATE_TEST_CASE_P(WorkingBackend, | 762 INSTANTIATE_TEST_CASE_P(WorkingBackend, |
| 763 PasswordStoreXTest, | 763 PasswordStoreXTest, |
| 764 testing::Values(WORKING_BACKEND)); | 764 testing::Values(WORKING_BACKEND)); |
| OLD | NEW |