WebCrypto: Implement digest() using NSS

content/renderer/webcrypto_sha_digest.h

+// Copyright (c) 2013 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.
+
+#ifndef CONTENT_RENDERER_WEBCRYPTO_SHA_DIGEST_H_
+#define CONTENT_RENDERER_WEBCRYPTO_SHA_DIGEST_H_
+
+#include "base/compiler_specific.h"
+
+#if defined(USE_NSS)
+#include <pk11pub.h>
+#endif

[Ryan Sleevi, 2013/07/31 18:19:19]

nit: Put this header *below* the other headers

See http://www.chromium.org/developers/coding-style#TOC-Platform-specific-code

However, see
http://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Minimiz...
- it would be better to forward declare PK11Context

[Bryan Eyler, 2013/08/02 00:49:05]

Changed to forward declaration.  It's not a very pretty declaration though...

+
+#include "third_party/WebKit/public/platform/WebCrypto.h"
+#include "third_party/WebKit/public/platform/WebCryptoAlgorithm.h"
+
+namespace content {
+
+class WebCryptoSHADigest : public WebKit::WebCryptoOperation {
+ public:
+  explicit WebCryptoSHADigest(const WebKit::WebCryptoAlgorithmId algorithm_id,
+                              WebKit::WebCryptoOperationResult& result);
+  virtual ~WebCryptoSHADigest();
+
+  bool Initialize();
+
+  virtual void process(const unsigned char* bytes, size_t size) OVERRIDE;
+  virtual void abort() OVERRIDE;
+  virtual void finish() OVERRIDE;
+
+ private:
+  WebKit::WebCryptoOperationResult& result_;

[eroman, 2013/07/31 02:23:50]

This needs to be a copy, otherwise it can end up being be a use-after-free.

[Bryan Eyler, 2013/08/02 00:49:05]

Done.

+
+#if defined(USE_NSS)
+  PK11Context* context_;
+  SECOidTag hash_algorithm_;
+#endif
+
+  unsigned int hash_result_length_;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_RENDERER_WEBCRYPTO_SHA_DIGEST_H_