[WebIDL] Remove custom conversion from FontFace code by using a Variant
Created attachment 301642 [details] Patch
Comment on attachment 301642 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=301642&action=review > Source/WebCore/css/FontFace.idl:47 > - Constructor(DOMString family, any source, optional FontFaceDescriptors descriptors) > + Constructor(DOMString family, (DOMString or BinaryData) source, optional FontFaceDescriptors descriptors) Did you used to be able to call this with JSValues that were not ArrayBuffers or ArrayBufferViews? Could you add a test that verifies that that can't be done any more?
(In reply to comment #2) > Comment on attachment 301642 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=301642&action=review > > > Source/WebCore/css/FontFace.idl:47 > > - Constructor(DOMString family, any source, optional FontFaceDescriptors descriptors) > > + Constructor(DOMString family, (DOMString or BinaryData) source, optional FontFaceDescriptors descriptors) > > Did you used to be able to call this with JSValues that were not > ArrayBuffers or ArrayBufferViews? Could you add a test that verifies that > that can't be done any more? Good idea. I think the old code was wrong, in that it should be converting anything that is not a ArrayBuffer or ArrayBufferView to a String. I'll add a test.
Committed r212405: <http://trac.webkit.org/changeset/212405>