[webkit-reviews] review granted: [Bug 123598] Add a Mac WebCrypto implementation of HMAC importKey/sign/verify : [Attachment 215694] proposed patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Oct 31 17:22:49 PDT 2013
Anders Carlsson <andersca at apple.com> has granted Alexey Proskuryakov
<ap at webkit.org>'s request for review:
Bug 123598: Add a Mac WebCrypto implementation of HMAC importKey/sign/verify
https://bugs.webkit.org/show_bug.cgi?id=123598
Attachment 215694: proposed patch
https://bugs.webkit.org/attachment.cgi?id=215694&action=review
------- Additional Comments from Anders Carlsson <andersca at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=215694&action=review
> Source/WebCore/bindings/js/JSSubtleCryptoCustom.cpp:235
> + std::unique_ptr<PromiseWrapper> promiseWrapper =
PromiseWrapper::create(globalObject(), promise);
I’d use auto for the variable declaration here.
> Source/WebCore/bindings/js/JSSubtleCryptoCustom.cpp:238
> + algorithm->verify(*parameters, *key.get(), signature, data,
std::move(promiseWrapper), ec);
Can’t you just do *key here?
> Source/WebCore/bindings/js/JSSubtleCryptoCustom.cpp:337
> + std::unique_ptr<PromiseWrapper> promiseWrapper =
PromiseWrapper::create(globalObject(), promise);
Auto?
> Source/WebCore/crypto/keys/CryptoKeyHMAC.cpp:52
> + std::unique_ptr<CryptoAlgorithmDescriptionBuilder>
hashDescriptionBuilder = builder.createEmptyClone();
Auto?
More information about the webkit-reviews
mailing list