Require extractable=false for KDF import

Require extractable=false for KDF import

This was added to the spec in:
https://github.com/w3c/webcrypto/commit/90d9f747da7ab2dd0f88638666c2326dcb953536

BUG=630025
Committed: https://crrev.com/c9529a8212e9ab0eff2679cbc20d8d5d1297b54f
Cr-Commit-Position: refs/heads/master@{#417016}

[eroman, 2016-08-30 01:57:34]

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

[commit-bot: I haz the power, 2016-08-30 02:47:03]

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-30 04:36:35]

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

[commit-bot: I haz the power, 2016-08-30 04:36:35]

Dry run: This issue passed the CQ dry run.

[eroman, 2016-09-06 23:35:39]

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

[commit-bot: I haz the power, 2016-09-06 23:35:57]

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

[eroman, 2016-09-06 23:37:49]

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

[commit-bot: I haz the power, 2016-09-07 01:15:49]

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

[commit-bot: I haz the power, 2016-09-07 01:15:52]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

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

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

[commit-bot: I haz the power, 2016-09-07 09:00:36]

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-09-07 10:58:11]

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

[commit-bot: I haz the power, 2016-09-07 10:58:12]

Dry run: This issue passed the CQ dry run.

[Ryan Sleevi, 2016-09-07 18:37:40]

lgtm

[eroman, 2016-09-07 18:48:44]

I failed to mention this in the CL description:

This change may cause some compatibility problems, since importKey() calls that
were working before may break.

In fact this was the case with some existing layout tests which were passing
extractable=true (having done so without much thought).

I don't have data on how widespread this is, and was planning to just go for it
and rely on the dev/beta channels to sound the alarm in time.

[Ryan Sleevi, 2016-09-07 18:51:36]

On 2016/09/07 18:48:44, eroman wrote:
> I failed to mention this in the CL description:
> 
> This change may cause some compatibility problems, since importKey() calls
that
> were working before may break.
> 
> In fact this was the case with some existing layout tests which were passing
> extractable=true (having done so without much thought).
> 
> I don't have data on how widespread this is, and was planning to just go for
it
> and rely on the dev/beta channels to sound the alarm in time.

Yeah, I saw that from the comment and bug, and still think it's the Right and
Consistent thing. But if you're concerned, you could always check with one of
the standards TLs to gather thoughts.

[eroman, 2016-09-07 18:53:52]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2016-09-07 18:54:41]

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-09-07 18:59:53]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-07 19:02:18]

Description was changed from

==========
Require extractable=false for KDF import

This was added to the spec in:
https://github.com/w3c/webcrypto/commit/90d9f747da7ab2dd0f88638666c2326dcb953536

BUG=630025
==========

to

==========
Require extractable=false for KDF import

This was added to the spec in:
https://github.com/w3c/webcrypto/commit/90d9f747da7ab2dd0f88638666c2326dcb953536

BUG=630025

Committed: https://crrev.com/c9529a8212e9ab0eff2679cbc20d8d5d1297b54f
Cr-Commit-Position: refs/heads/master@{#417016}
==========

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

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