[webcrypto] Check for empty key usages *after* key creation rather than before, to match the spec's…

[webcrypto] Check for empty key usages *after* key creation rather than before, to match the spec's order of failures.

This affects:
  * importKey()
  * generateKey()
  * unwrapKey()

This also removes a "fail-fast optimization" on key usage when importing JWK asymmetric keys, as that was not supported by the spec.

BUG=611801
Committed: https://crrev.com/0bb976e4a86cd456a35d861e7fb7f9e9a729a1a5
Cr-Commit-Position: refs/heads/master@{#409471}

[eroman, 2016-07-19 22:01:46]

The CQ bit was checked by eroman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-19 22:03:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[eroman, 2016-07-19 22:09:13]

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

[eroman, 2016-07-19 22:09:22]

The CQ bit was checked by eroman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-19 22:10:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-19 23:38:35]

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

[commit-bot: I haz the power, 2016-07-19 23:38:36]

Dry run: This issue passed the CQ dry run.

[eroman, 2016-08-01 20:38:43]

Ping. Still no rush on this, but just a reminder

[Ryan Sleevi, 2016-08-03 01:40:22]

lgtm

[eroman, 2016-08-03 01:46:08]

The CQ bit was checked by eroman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-03 01:46:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-03 03:24:14]

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

[commit-bot: I haz the power, 2016-08-03 03:24:15]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[eroman, 2016-08-03 05:57:54]

The CQ bit was checked by eroman@chromium.org

[eroman, 2016-08-03 05:57:54]

The patchset sent to the CQ was uploaded after l-g-t-m from rsleevi@chromium.org
Link to the patchset: https://codereview.chromium.org/2163053002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2016-08-03 05:58:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-03 06:36:58]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-03 06:38:43]

Description was changed from

==========
[webcrypto] Check for empty key usages *after* key creation rather than before,
to match the spec's order of failures.

This affects:
  * importKey()
  * generateKey()
  * unwrapKey()

This also removes a "fail-fast optimization" on key usage when importing JWK
asymmetric keys, as that was not supported by the spec.

BUG=611801
==========

to

==========
[webcrypto] Check for empty key usages *after* key creation rather than before,
to match the spec's order of failures.

This affects:
  * importKey()
  * generateKey()
  * unwrapKey()

This also removes a "fail-fast optimization" on key usage when importing JWK
asymmetric keys, as that was not supported by the spec.

BUG=611801

Committed: https://crrev.com/0bb976e4a86cd456a35d861e7fb7f9e9a729a1a5
Cr-Commit-Position: refs/heads/master@{#409471}
==========

[commit-bot: I haz the power, 2016-08-03 06:38:44]

Patchset 3 (id:??) landed as
https://crrev.com/0bb976e4a86cd456a35d861e7fb7f9e9a729a1a5
Cr-Commit-Position: refs/heads/master@{#409471}