The other conditions for generating visitChildren() are adequate if the class needs its own visitChildren. Currently, we're generating some visitChildren() methods that only call its Base::visitChildren(). Removing the "is / extends EventTarget" conditions removes those unneeded visitChildren() overrides.
<rdar://problem/32498266>
Ditto for the IsJSBuiltinConstructor condition, which is not used to emit any additional code in the visitChildren override.
Created attachment 311651 [details] proposed patch. Let's test this on the EWS first.
Comment on attachment 311651 [details] proposed patch. r=me as long as bots are happy.
Thanks for the review.
Comment on attachment 311651 [details] proposed patch. Clearing flags on attachment: 311651 Committed r217642: <http://trac.webkit.org/changeset/217642>
All reviewed patches have been landed. Closing bug.