[webcrypto] Refactor RSA-SSA signing.

[webcrypto] Refactor RSA-SSA signing.

This extracts the signing/verification functions, so the subsequent RSA-PSS changelist is easier to follow.

BUG=399090
Committed: https://crrev.com/72ce4c6db27260e429a2269cf4ddcb59e9ba2581
Cr-Commit-Position: refs/heads/master@{#300194}

[eroman, 2014-10-15 23:44:41]

eroman@chromium.org changed reviewers:
+ rsleevi@chromium.org

[Ryan Sleevi, 2014-10-17 21:09:56]

LGTM % nit

https://codereview.chromium.org/662533002/diff/1/content/child/webcrypto/open...
File content/child/webcrypto/openssl/rsa_sign_openssl.cc (right):

https://codereview.chromium.org/662533002/diff/1/content/child/webcrypto/open...
content/child/webcrypto/openssl/rsa_sign_openssl.cc:90: if
(!EVP_DigestVerifyUpdate(ctx.get(), data.bytes(), data.byte_length())) {
style: why braces here, but not on the other two liners?

[eroman, 2014-10-17 22:49:17]

https://codereview.chromium.org/662533002/diff/1/content/child/webcrypto/open...
File content/child/webcrypto/openssl/rsa_sign_openssl.cc (right):

https://codereview.chromium.org/662533002/diff/1/content/child/webcrypto/open...
content/child/webcrypto/openssl/rsa_sign_openssl.cc:90: if
(!EVP_DigestVerifyUpdate(ctx.get(), data.bytes(), data.byte_length())) {
On 2014/10/17 21:09:55, Ryan Sleevi wrote:
> style: why braces here, but not on the other two liners?

Done -- Removed the brace.

[eroman, 2014-10-17 23:35:51]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2014-10-17 23:37:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/662533002/20001

[commit-bot: I haz the power, 2014-10-18 01:12:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-18 01:13:23]

Patchset 2 (id:??) landed as
https://crrev.com/72ce4c6db27260e429a2269cf4ddcb59e9ba2581
Cr-Commit-Position: refs/heads/master@{#300194}