[webcrypto] Fix AES-KW unwrapping for symmetric keys (NSS).

[webcrypto] Fix AES-KW unwrapping for symmetric keys (NSS).

The unwrapped key did not have the proper flags so it could not be used in other operations (like signing).

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

[Ryan Sleevi, 2014-05-08 00:58:43]

lgtm

https://codereview.chromium.org/269313004/diff/1/content/child/webcrypto/shar...
File content/child/webcrypto/shared_crypto_unittest.cc (right):

https://codereview.chromium.org/269313004/diff/1/content/child/webcrypto/shar...
content/child/webcrypto/shared_crypto_unittest.cc:2800: // Unwrap a HMAC key
using AES-KW, and then try signing using the unwrapped key.
"then try doing a sign/verify with the unwrapped key"

[eroman, 2014-05-08 01:02:39]

https://codereview.chromium.org/269313004/diff/1/content/child/webcrypto/shar...
File content/child/webcrypto/shared_crypto_unittest.cc (right):

https://codereview.chromium.org/269313004/diff/1/content/child/webcrypto/shar...
content/child/webcrypto/shared_crypto_unittest.cc:2800: // Unwrap a HMAC key
using AES-KW, and then try signing using the unwrapped key.
On 2014/05/08 00:58:44, Ryan Sleevi wrote:
> "then try doing a sign/verify with the unwrapped key"

Done.

[eroman, 2014-05-08 01:02:43]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2014-05-08 01:06:40]

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

[eroman, 2014-05-08 05:01:26]

The CQ bit was unchecked by eroman@chromium.org

[eroman, 2014-05-08 05:01:32]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2014-05-08 05:02:24]

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

[commit-bot: I haz the power, 2014-05-08 06:04:59]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_dbg on tryserver.chromium
  linux_chromium_chromeos_rel on tryserver.chromium
  linux_chromium_rel on tryserver.chromium

[commit-bot: I haz the power, 2014-05-08 06:16:56]

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

[commit-bot: I haz the power, 2014-05-08 06:16:57]

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

[eroman, 2014-05-08 17:52:14]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2014-05-08 18:00:03]

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

[commit-bot: I haz the power, 2014-05-09 13:34:47]

Change committed as 269262