[webcrypto] Add length parameter to HmacKeyAlgorithm. (chromium)

[webcrypto] Add length parameter to HmacKeyAlgorithm. (chromium)

Matches a planned change to the spec:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=25094

Blink side of this change: https://codereview.chromium.org/204013006/

BUG=245025
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=257878

[eroman, 2014-03-18 23:50:01]

https://codereview.chromium.org/203303006/diff/20001/content/child/webcrypto/...
File content/child/webcrypto/webcrypto_util.cc (right):

https://codereview.chromium.org/203303006/diff/20001/content/child/webcrypto/...
content/child/webcrypto/webcrypto_util.cc:390: new
blink::WebCryptoHmacKeyAlgorithmParams(hash, keylen_bytes * 8));
I didn't check for overflow, probably should. wdyt?

[eroman, 2014-03-19 00:04:19]

https://codereview.chromium.org/203303006/diff/20001/content/child/webcrypto/...
File content/child/webcrypto/webcrypto_util.cc (right):

https://codereview.chromium.org/203303006/diff/20001/content/child/webcrypto/...
content/child/webcrypto/webcrypto_util.cc:390: new
blink::WebCryptoHmacKeyAlgorithmParams(hash, keylen_bytes * 8));
On 2014/03/18 23:50:01, eroman wrote:
> I didn't check for overflow, probably should. wdyt?

OK done.

[Ryan Sleevi, 2014-03-19 00:07:27]

lgtm

[eroman, 2014-03-19 00:10:54]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2014-03-19 00:13:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eroman@chromium.org/203303006/40001

[commit-bot: I haz the power, 2014-03-19 00:21:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-19 00:21:21]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_clang_dbg

[eroman, 2014-03-19 00:35:29]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2014-03-19 00:39:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eroman@chromium.org/203303006/40001

[commit-bot: I haz the power, 2014-03-19 01:09:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-19 01:09:34]

Try jobs failed on following builders:
  tryserver.chromium on win_chromium_compile_dbg

[eroman, 2014-03-19 01:39:36]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2014-03-19 01:42:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eroman@chromium.org/203303006/40001

[commit-bot: I haz the power, 2014-03-19 03:06:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-19 03:06:30]

Try jobs failed on following builders:
  tryserver.chromium on mac_chromium_rel

[eroman, 2014-03-19 03:37:01]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2014-03-19 03:38:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eroman@chromium.org/203303006/40001

[commit-bot: I haz the power, 2014-03-19 06:39:00]

Change committed as 257878