Summary: | iterable<> should be enabled on WK1 | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||||
Component: | DOM | Assignee: | Chris Dumez <cdumez> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | cdumez, commit-queue, mmaxfield, webkit-bug-importer, youennf | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Chris Dumez
2017-01-19 16:51:49 PST
This means `for ... of` and forEach() do not work on iterable types such as NodeList. Created attachment 299288 [details]
Patch
Comment on attachment 299288 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=299288&action=review > Source/WebCore/bindings/scripts/test/TestNode.idl:-28 > - [EnabledAtRuntime=DOMIterator] iterable<TestNode>; Are there cases where we will like to make utterable enabledatruntime with another flag? If so, it is best to keep TestNode.idl as is. Otherwise, we might want to update the binding generator, addIterablePrototypeProperties in particular. Created attachment 299295 [details]
Patch
Created attachment 299297 [details]
Patch
Comment on attachment 299297 [details]
Patch
I'll cq once EWS bots will be happy about it
Comment on attachment 299297 [details] Patch Clearing flags on attachment: 299297 Committed r210949: <http://trac.webkit.org/changeset/210949> All reviewed patches have been landed. Closing bug. |