Cryptohome: Notify about error in async calls if cryptohome is not ready yet.

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 816c106b25cef1f53491a8edcef736827a2374ce..0e6f657069f89ecabccc2b0d681730ec2aa8e060 100644
--- a/chromeos/dbus/cryptohome_client.h
+++ b/chromeos/dbus/cryptohome_client.h
@@ -34,6 +34,10 @@ class UpdateKeyRequest;
 
 namespace chromeos {
 
+// Constant that will be passed to AsyncMethodCallback to indicate that
+// cryptohome is not ready yet.
+extern const int kCryptohomeNotReadyAsyncId;

[Daniel Erat, 2015/01/28 18:01:46]

can you make this be a public static member of CryptohomeClient (with a shorter
name) instead of putting it directly into the cryptohome namespace? that makes
it easier to see where it's defined when reading code that uses it.

e.g.

class ... CryptohomeClient ... {
 public:
  static const int kNotReadyAsyncId;

[Denis Kuznetsov (DE-MUC), 2015/01/28 19:45:26]

Done.

+
 // CryptohomeClient is used to communicate with the Cryptohome service.
 // All method should be called from the origin thread (UI thread) which
 // initializes the DBusThreadManager instance.