RESOLVED FIXED 173513
[WebIDL] Properly model buffer source / typed arrays as their own IDL types
https://bugs.webkit.org/show_bug.cgi?id=173513
Summary [WebIDL] Properly model buffer source / typed arrays as their own IDL types
Sam Weinig
Reported 2017-06-17 10:52:32 PDT
[WebIDL] Properly model buffer source / typed arrays as their own IDL types
Attachments
Patch (72.59 KB, patch)
2017-06-17 11:13 PDT, Sam Weinig
no flags
Archive of layout-test-results from ews101 for mac-elcapitan (1.40 MB, application/zip)
2017-06-17 12:24 PDT, Build Bot
no flags
Archive of layout-test-results from ews107 for mac-elcapitan-wk2 (1.50 MB, application/zip)
2017-06-17 12:29 PDT, Build Bot
no flags
Archive of layout-test-results from ews125 for ios-simulator-wk2 (10.94 MB, application/zip)
2017-06-17 12:50 PDT, Build Bot
no flags
Archive of layout-test-results from ews116 for mac-elcapitan (2.00 MB, application/zip)
2017-06-17 12:54 PDT, Build Bot
no flags
Patch (1.92 MB, patch)
2017-06-17 13:05 PDT, Sam Weinig
no flags
Patch (1.92 MB, patch)
2017-06-17 14:03 PDT, Sam Weinig
no flags
Sam Weinig
Comment 1 2017-06-17 11:13:43 PDT Comment hidden (obsolete)
Build Bot
Comment 2 2017-06-17 12:24:14 PDT Comment hidden (obsolete)
Build Bot
Comment 3 2017-06-17 12:24:15 PDT Comment hidden (obsolete)
Build Bot
Comment 4 2017-06-17 12:29:26 PDT Comment hidden (obsolete)
Build Bot
Comment 5 2017-06-17 12:29:28 PDT Comment hidden (obsolete)
Build Bot
Comment 6 2017-06-17 12:50:23 PDT Comment hidden (obsolete)
Build Bot
Comment 7 2017-06-17 12:50:24 PDT Comment hidden (obsolete)
Build Bot
Comment 8 2017-06-17 12:54:31 PDT Comment hidden (obsolete)
Build Bot
Comment 9 2017-06-17 12:54:33 PDT Comment hidden (obsolete)
Sam Weinig
Comment 10 2017-06-17 13:05:51 PDT Comment hidden (obsolete)
Build Bot
Comment 11 2017-06-17 13:10:23 PDT Comment hidden (obsolete)
Sam Weinig
Comment 12 2017-06-17 14:03:37 PDT
Sam Weinig
Comment 13 2017-06-17 14:39:45 PDT
Reviewers: Don't be put off by the size of the patch. The vast majority of that comes from one huge test that mechanically had a lot exception strings changed.
Alex Christensen
Comment 14 2017-06-19 09:14:02 PDT
Comment on attachment 313212 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=313212&action=review > Source/WebCore/bindings/js/JSDOMConvertUnion.h:38 > +struct ConditionalReturner; Interesting. I feel like there might be a better way to do this.
WebKit Commit Bot
Comment 15 2017-06-19 09:47:18 PDT
Comment on attachment 313212 [details] Patch Clearing flags on attachment: 313212 Committed r218495: <http://trac.webkit.org/changeset/218495>
WebKit Commit Bot
Comment 16 2017-06-19 09:47:20 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.