RESOLVED FIXED 154577
Remove IteratorKey and IteratorValue declarations from JSXX class declarations.
https://bugs.webkit.org/show_bug.cgi?id=154577
Summary Remove IteratorKey and IteratorValue declarations from JSXX class declarations.
youenn fablet
Reported 2016-02-23 01:49:11 PST
Based on bug 154531, we no longer need these type declarations.
Attachments
Patch (2.55 KB, patch)
2016-02-23 01:52 PST, youenn fablet
no flags
Patch for landing (3.08 KB, patch)
2016-02-24 05:19 PST, youenn fablet
no flags
youenn fablet
Comment 1 2016-02-23 01:52:37 PST
Myles C. Maxfield
Comment 2 2016-02-23 11:27:32 PST
Comment on attachment 271998 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=271998&action=review > Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:-1062 > - if ($interface->iterable) { > - push(@headerContent, "\n"); > - if ($interface->iterable->keyType) { > - my $keyType = GetNativeType($interface->iterable->keyType); > - push(@headerContent, " using IteratorKey = $keyType;\n"); > - } > - my $valueType = GetNativeType($interface->iterable->valueType); > - push(@headerContent, " using IteratorValue = $valueType;\n"); > - } > - Please delete the line from GetNativeType() that special-cases FontFace too.
youenn fablet
Comment 3 2016-02-24 05:19:28 PST
Created attachment 272112 [details] Patch for landing
youenn fablet
Comment 4 2016-02-24 05:20:29 PST
Thanks for the review. > Please delete the line from GetNativeType() that special-cases FontFace too. Done in the landing patch.
WebKit Commit Bot
Comment 5 2016-02-24 06:17:45 PST
Comment on attachment 272112 [details] Patch for landing Clearing flags on attachment: 272112 Committed r197028: <http://trac.webkit.org/changeset/197028>
WebKit Commit Bot
Comment 6 2016-02-24 06:17:49 PST
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.