From 75d112d2ab997e2e998007733602c7d1ca72f9ff Mon Sep 17 00:00:00 2001 From: "stuartmorgan@chromium.org" Date: Thu, 2 Jul 2009 16:58:03 +0000 Subject: Implement Add and Update for PasswordStoreMac. Modify LoginDatabase slightly to give PasswordStoreMac enough information to do the right thing. Add creator code for keychain items we create, and unit tests to make sure. BUG=11745 TEST=Visit a site for which you have a password in the Keychain. Type your username, unfocus the field, and then log in with the filled password. Log out, return to the login page, and the username and password should now autofill without user interaction. Review URL: http://codereview.chromium.org/151176 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19822 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/password_manager/login_database_unittest.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'chrome/browser/password_manager/login_database_unittest.cc') diff --git a/chrome/browser/password_manager/login_database_unittest.cc b/chrome/browser/password_manager/login_database_unittest.cc index 76dce57..8b4f6a9 100644 --- a/chrome/browser/password_manager/login_database_unittest.cc +++ b/chrome/browser/password_manager/login_database_unittest.cc @@ -159,7 +159,9 @@ TEST_F(LoginDatabaseTest, Logins) { // We update, and check to make sure it matches the // old form, and there is only one record. - EXPECT_TRUE(db->UpdateLogin(form6)); + int rows_changed = 0; + EXPECT_TRUE(db->UpdateLogin(form6, &rows_changed)); + EXPECT_EQ(1, rows_changed); // matches EXPECT_TRUE(db->GetLogins(form5, &result)); EXPECT_EQ(1U, result.size()); -- cgit v1.1