Add support for the Pkcs11GetTpmTokenInfoForUser cryptohome call.

chromeos/dbus/cryptohome_client.h

Index: chromeos/dbus/cryptohome_client.h
diff --git a/chromeos/dbus/cryptohome_client.h b/chromeos/dbus/cryptohome_client.h
index b1019c5b7b1f6cdd02c88f04cdd25bdfb6684a19..5444376d7fbf6965a7423d175e0c65ee681deb3c 100644
--- a/chromeos/dbus/cryptohome_client.h
+++ b/chromeos/dbus/cryptohome_client.h
@@ -39,7 +39,8 @@ class CHROMEOS_EXPORT CryptohomeClient : public DBusClient {
   typedef base::Callback<void(
       DBusMethodCallStatus call_status,
       const std::string& label,
-      const std::string& user_pin)> Pkcs11GetTpmTokenInfoCallback;
+      const std::string& user_pin,
+      int slot)> Pkcs11GetTpmTokenInfoCallback;

[satorux1, 2013/10/08 04:29:33]

What is slot? could you add some comments about label, user_bin, and slot while
you are at it?

[Darren Krahn, 2013/10/08 17:06:41]

Done.

   // A callback for methods which return both a bool result and data.
   typedef base::Callback<void(DBusMethodCallStatus call_status,
                               bool result,
@@ -190,6 +191,10 @@ class CHROMEOS_EXPORT CryptohomeClient : public DBusClient {
   virtual void Pkcs11GetTpmTokenInfo(
       const Pkcs11GetTpmTokenInfoCallback& callback) = 0;
 
+  virtual void Pkcs11GetTpmTokenInfoForUser(

[satorux1, 2013/10/08 04:29:33]

Function comment is missing.  "Calls XXX method." is not particularly useful,
but might be better than nothing for consistency. :P 

I think it'd be nice to describe the difference between this function and
Pkcs11GetTpmTokenInfo, which looks very similar.

[Darren Krahn, 2013/10/08 17:06:41]

Done.

+      const std::string& username,
+      const Pkcs11GetTpmTokenInfoCallback& callback) = 0;
+
   // Calls InstallAttributesGet method and returns true when the call succeeds.
   // This method blocks until the call returns.
   // The original content of |value| is lost.