RESOLVED FIXED 169232
[WebCrypto] Make sure all CryptoKey classes are structured clonable 
https://bugs.webkit.org/show_bug.cgi?id=169232
Summary [WebCrypto] Make sure all CryptoKey classes are structured clonable
Jiewen Tan
Reported 2017-03-06 15:59:37 PST
Make CryptoKeyEC structured clonable.
Attachments
Patch (31.03 KB, patch)
2017-03-16 20:57 PDT, Jiewen Tan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jiewen Tan
Comment 1 2017-03-06 16:00:52 PST
<rdar://problem/23789585>
Jiewen Tan
Comment 2 2017-03-09 00:32:13 PST
(In reply to comment #1) > <rdar://problem/23789585> This radar is faked.
Jiewen Tan
Comment 3 2017-03-09 00:33:06 PST
We should make sure all CryptoKey classes are structured clonable.
Jiewen Tan
Comment 4 2017-03-09 00:35:36 PST
This includes CryptoKeyAES, CryptoKeyHMAC, CryptoKeyEE, CryptoKeyRSA and CryptoKeyRaw.
Jiewen Tan
Comment 5 2017-03-09 00:37:06 PST
For all CryptoKey, we should make sure they have move constructors.
Jiewen Tan
Comment 6 2017-03-16 20:47:00 PDT
<rdar://problem/31106660>
Jiewen Tan
Comment 7 2017-03-16 20:57:41 PDT
Created attachment 304747 [details] Patch
WebKit Commit Bot
Comment 8 2017-03-16 20:59:50 PDT
Attachment 304747 [details] did not pass style-queue: ERROR: Source/WebCore/bindings/js/SerializedScriptValue.cpp:196: enum members should use InterCaps with an initial capital letter or initial 'k' for C-style enums. [readability/enum_casing] [4] ERROR: Source/WebCore/bindings/js/SerializedScriptValue.cpp:197: enum members should use InterCaps with an initial capital letter or initial 'k' for C-style enums. [readability/enum_casing] [4] Total errors found: 2 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
Brent Fulgham
Comment 9 2017-03-17 13:18:29 PDT
Comment on attachment 304747 [details] Patch r=me.
Jiewen Tan
Comment 10 2017-03-17 13:48:59 PDT
Comment on attachment 304747 [details] Patch Thanks Brent for r+ my patch.
WebKit Commit Bot
Comment 11 2017-03-17 14:18:46 PDT
Comment on attachment 304747 [details] Patch Clearing flags on attachment: 304747 Committed r214122: <http://trac.webkit.org/changeset/214122>
WebKit Commit Bot
Comment 12 2017-03-17 14:18:50 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.