Password Login Database: report correct changes from UpdateLogin().

chrome/browser/password_manager/native_backend_kwallet_x_unittest.cc

Index: chrome/browser/password_manager/native_backend_kwallet_x_unittest.cc
diff --git a/chrome/browser/password_manager/native_backend_kwallet_x_unittest.cc b/chrome/browser/password_manager/native_backend_kwallet_x_unittest.cc
index 1b3e96624787d7bd54717ca9c5f4b9891f7d8ad6..fbeb230b4487b7b997ae0e21f8b19d9fd0acd80e 100644
--- a/chrome/browser/password_manager/native_backend_kwallet_x_unittest.cc
+++ b/chrome/browser/password_manager/native_backend_kwallet_x_unittest.cc
@@ -540,6 +540,42 @@ TEST_F(NativeBackendKWalletTest, BasicAddLogin) {
   CheckPasswordForms("Chrome Form Data (42)", expected);
 }
 
+TEST_F(NativeBackendKWalletTest, BasicUpdateLogin) {

[Garrett Casto, 2014/05/21 19:14:20]

As with the other test, can you check that updating a non-matching login doesn't
add?

[vasilii, 2014/05/22 11:38:14]

Done.

+  NativeBackendKWalletStub backend(42);
+  EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
+
+  BrowserThread::PostTask(
+      BrowserThread::DB, FROM_HERE,
+      base::Bind(base::IgnoreResult(&NativeBackendKWalletStub::AddLogin),
+                 base::Unretained(&backend), form_google_));
+
+  RunDBThread();
+
+  PasswordForm new_form_google(form_google_);
+  new_form_google.times_used = 10;
+  new_form_google.action = GURL("http://www.google.com/different/login");
+
+  // Update login
+  PasswordStoreChangeList changes;
+  BrowserThread::PostTask(
+      BrowserThread::DB, FROM_HERE,
+      base::Bind(base::IgnoreResult(&NativeBackendKWalletStub::UpdateLogin),
+                 base::Unretained(&backend),
+                 new_form_google,
+                 base::Unretained(&changes)));
+  RunDBThread();
+
+  ASSERT_EQ(1u, changes.size());
+  EXPECT_EQ(PasswordStoreChange::UPDATE, changes.front().type());
+  EXPECT_EQ(new_form_google, changes.front().form());
+
+  std::vector<const PasswordForm*> forms;
+  forms.push_back(&new_form_google);
+  ExpectationArray expected;
+  expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
+  CheckPasswordForms("Chrome Form Data (42)", expected);
+}
+
 TEST_F(NativeBackendKWalletTest, BasicListLogins) {
   NativeBackendKWalletStub backend(42);
   EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));