Added serialization to SecureHash.

crypto/secure_hash.h

Index: crypto/secure_hash.h
diff --git a/crypto/secure_hash.h b/crypto/secure_hash.h
index 8a0f6ae66e1ea7f4edea01120ab42e70e73add1a..bff6600535850369c9a932c82be53a17b13200ce 100644
--- a/crypto/secure_hash.h
+++ b/crypto/secure_hash.h
@@ -9,6 +9,8 @@
 #include "base/basictypes.h"
 #include "crypto/crypto_export.h"
 
+class Pickle;
+
 namespace crypto {
 
 // A wrapper to calculate secure hashes incrementally, allowing to
@@ -25,6 +27,16 @@ class CRYPTO_EXPORT SecureHash {
   virtual void Update(const void* input, size_t len) = 0;
   virtual void Finish(void* output, size_t len) = 0;
 
+  // Serialize the context, so it can be restored at a later time.
+  // |pickle| will contain the serialized data.
+  // Returns whether or not |pickle| was filled.
+  virtual bool Serialize(Pickle* pickle) = 0;
+
+  // Restore the context that was saved earlier.

[Randy Smith (Not in Mondays), 2011/11/18 19:01:47]

Maybe worthwhile indicating that it will wipe out any in-progress context?

I find myself wondering if this could be replaced by a constructor taking a
Pickle*, but my guess is that that goes against the way Pickle is used in most
places, so it's probably not a good idea.  A related thing that might or might
not be a good idea would be to make a version of the Create() method that takes
a Pickle and automatically creates the type of SecureHash that it represents;
something to chat with the crypto owners about.

[ahendrickson, 2011/11/18 19:47:00]

I don't believe that we can currently have both NSS and OpenSSL hashes in the
same build, although it looks like we're on the way to supporting both.  The
Create() code takes the same parameter in both implementations, it's just that
only one of them is built at a time.  See crbug 62803.

There is a flag in build/common.gyp called use_openssl.

+  // |pickle| holds the saved data.
+  // Returns success or failure.
+  virtual bool Deserialize(Pickle* pickle) = 0;
+
  protected:
   SecureHash() {}