Fill empty fields from keyring result with default values.

chrome/browser/password_manager/native_backend_libsecret_unittest.cc

 // Copyright (c) 2015 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/prefs/pref_service.h"
 #include "base/stl_util.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 30 matching lines @@
 struct MockSecretItem {
   MockSecretValue* value;
   GHashTable* attributes;
 
   MockSecretItem(MockSecretValue* value, GHashTable* attributes)
       : value(value), attributes(attributes) {}
   ~MockSecretItem() {
     delete value;
     g_hash_table_destroy(attributes);
   }
+
+  void RemoveAttribute(const char* keyname) {
+    g_hash_table_remove(attributes, keyname);
+  }
 };
 
 bool Matches(MockSecretItem* item, GHashTable* query) {
   GHashTable* attributes = item->attributes;
   GHashTableIter iter;
   gchar* name;
   gchar* query_value;
   g_hash_table_iter_init(&iter, query);
 
   while (g_hash_table_iter_next(&iter, reinterpret_cast<gpointer*>(&name),
@@ skipping 750 matching lines @@
 }
 
 TEST_F(NativeBackendLibsecretTest, RemoveLoginsCreatedBetween) {
   CheckRemoveLoginsBetween(CREATED);
 }
 
 TEST_F(NativeBackendLibsecretTest, RemoveLoginsSyncedBetween) {
   CheckRemoveLoginsBetween(SYNCED);
 }
 
+TEST_F(NativeBackendLibsecretTest, SomeKeyringAttributesAreMissing) {
+  // Absent attributes should be filled with default values.
+  NativeBackendLibsecret backend(42);
+
+  backend.AddLogin(form_google_);
+
+  EXPECT_EQ(1u, global_mock_libsecret_items.size());
+  // Remove a string attribute.
+  global_mock_libsecret_items[0]->RemoveAttribute("avatar_url");
+  // Remove an integer attribute.
+  global_mock_libsecret_items[0]->RemoveAttribute("ssl_valid");
+
+  ScopedVector<autofill::PasswordForm> form_list;
+  backend.GetAutofillableLogins(&form_list);
+
+  EXPECT_EQ(1u, form_list.size());
+  EXPECT_EQ(GURL(""), form_list[0]->avatar_url);
+  EXPECT_FALSE(form_list[0]->ssl_valid);
+}
+
 // TODO(mdm): add more basic tests here at some point.