Use size_t as the type of key_length and digest_length arguments of HMAC.

Use size_t as the type of the key_length and digest_length arguments
of HMAC::Init() and HMAC::Sign().

R=agl@chromium.org,bradnelson@chromium.org,thakis@chromium.org
BUG=none
TEST=none

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

[wtc, 2012-12-01 01:39:23]

agl: please review the entire CL. I noticed this problem when
I reviewed your HMAC CL.

thakis: please review chrome/browser/internal_auth.cc.

bradnelson: please review chrome/nacl/nacl_validation_query.h.

[Nico, 2012-12-01 01:47:37]

internal_auth lgtm

Trybots look fairly red though.

[agl, 2012-12-03 14:48:01]

lgtm

https://codereview.chromium.org/11419270/diff/2006/crypto/hmac_openssl.cc
File crypto/hmac_openssl.cc (right):

https://codereview.chromium.org/11419270/diff/2006/crypto/hmac_openssl.cc#new...
crypto/hmac_openssl.cc:46: DCHECK_NE(digest_length, 0u);
This change causes digest_length==0 to be rejected, but that doesn't seem to be
reflected in other platform implementations. Perhaps this DCHECK should be
removed?

[bradn, 2012-12-03 17:52:43]

LGTM on nacl files.

[commit-bot: I haz the power, 2012-12-03 18:55:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wtc@chromium.org/11419270/3005

[commit-bot: I haz the power, 2012-12-04 00:50:37]

Change committed as 170852