Only reauthorize passwords. Don't reauthorize private keys.

chrome/browser/mac/keychain_reauthorize.mm

Index: chrome/browser/mac/keychain_reauthorize.mm
===================================================================
--- chrome/browser/mac/keychain_reauthorize.mm	(revision 139070)
+++ chrome/browser/mac/keychain_reauthorize.mm	(working copy)
@@ -455,6 +455,22 @@
     return;
   }
 
+  // CrSKeychainItemCreateFromContent (SecKeychainItemCreateFromContent)
+  // returns errKCNoSuchAttr (errSecNoSuchAttr) when asked to add an item of
+  // type kSecPrivateKeyItemClass. This would happen after the original
+  // private key was deleted, resulting in data loss. I can't figure out how
+  // SecKeychainItemCreateFromContent wants private keys added. Skip them,
+  // only doing the reauthorization for Keychain item types known to work,
+  // the item types expected to be used by most users and those that are
+  // synced.

[Nico, 2012/06/01 19:09:14]

nit: Add a http://crbug.com/ link to the comment

+  switch (old_attributes_and_data.item_class()) {
+    case kSecInternetPasswordItemClass:
+    case kSecGenericPasswordItemClass:
+      break;
+    default:
+      return;
+  }
+
   // SecKeychainItemCreateFromContent fails if any attribute is zero-length,
   // but old_attributes_and_data can contain zero-length attributes. Create
   // a new attribute list devoid of zero-length attributes.