Linux: support append to password list in the GNOME keyring backend to fix initial password sync.

chrome/browser/password_manager/native_backend_gnome_x_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stdarg.h>
 
 #include "base/basictypes.h"
 #include "base/stl_util.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
@@ skipping 521 matching lines @@
                         &NativeBackendGnome::AddLogin,
                         form_google_));
 
   RunBothThreads();
 
   EXPECT_EQ(1u, mock_keyring_items.size());
   if (mock_keyring_items.size() > 0)
     CheckMockKeyringItem(&mock_keyring_items[0], form_google_, "chrome-42");
 }
 
+TEST_F(NativeBackendGnomeTest, ListLoginsAppends) {
+  // Pretend that the migration has already taken place.
+  profile_->GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, true);
+
+  NativeBackendGnome backend(42, profile_->GetPrefs());
+  backend.Init();
+
+  BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
+      NewRunnableMethod(&backend,
+                        &NativeBackendGnome::AddLogin,
+                        form_google_));
+
+  // Send the same request twice with the same list both times.
+  std::vector<PasswordForm*> form_list;
+  BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
+      NewRunnableMethod(&backend,
+                        &NativeBackendGnome::GetAutofillableLogins,
+                        &form_list));
+  BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
+      NewRunnableMethod(&backend,
+                        &NativeBackendGnome::GetAutofillableLogins,
+                        &form_list));
+
+  RunBothThreads();
+
+  // Quick check that we got two results back.
+  EXPECT_EQ(2u, form_list.size());
+  STLDeleteElements(&form_list);
+
+  EXPECT_EQ(1u, mock_keyring_items.size());
+  if (mock_keyring_items.size() > 0)
+    CheckMockKeyringItem(&mock_keyring_items[0], form_google_, "chrome-42");
+}
+
 // TODO(mdm): add more basic (i.e. non-migration) tests here at some point.
 
 TEST_F(NativeBackendGnomeTest, MigrateOneLogin) {
   // Reject attempts to migrate so we can populate the store.
   mock_keyring_reject_local_ids = true;
 
   {
     NativeBackendGnome backend(42, profile_->GetPrefs());
     backend.Init();
 
@@ skipping 308 matching lines @@
     RunBothThreads();
 
     // The other two copies of the password in different profiles should remain.
     EXPECT_EQ(2u, mock_keyring_items.size());
     if (mock_keyring_items.size() > 0)
       CheckMockKeyringItem(&mock_keyring_items[0], form_google_, "chrome");
     if (mock_keyring_items.size() > 1)
       CheckMockKeyringItem(&mock_keyring_items[1], form_google_, "chrome-42");
   }
 }