Clean-up remained functions of HTML elements to auto-generated isFooElement() and toFooElement() helpers. Besides this patch clean-up unnecessary checks which are being supported by auto-generated isFooElement().
Created attachment 211174 [details] Patch
Created attachment 211175 [details] Patch
CC'ing Kling.
Comment on attachment 211175 [details] Patch Attachment 211175 [details] did not pass qt-wk2-ews (qt-wk2): Output: http://webkit-queues.appspot.com/results/1752009
Comment on attachment 211175 [details] Patch Attachment 211175 [details] did not pass qt-ews (qt): Output: http://webkit-queues.appspot.com/results/1743138
Comment on attachment 211175 [details] Patch Attachment 211175 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/1761010
Created attachment 211180 [details] Patch
Unfortunately we need to make toFooElement() use static_cast instead of reinterpret_cast as mentioned here: https://bugs.webkit.org/show_bug.cgi?id=120676#c8
(In reply to comment #8) > Unfortunately we need to make toFooElement() use static_cast instead of reinterpret_cast as mentioned here: > https://bugs.webkit.org/show_bug.cgi?id=120676#c8 Kling, yes, I talked it with kent on blink. But, it looks it is not easy. I need to have time to change it. *ElementTypeHelper.h may need to include all *Element.h.
Comment on attachment 211180 [details] Patch No request until solving reinterpret_cast<> problem.
Sure, I opened bug 121087 with a patch so we can at least have correct C++ until a better solution is found :)
Comment on attachment 211180 [details] Patch Attachment 211180 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/1735497
Created attachment 211272 [details] Patch
Comment on attachment 211272 [details] Patch Clearing flags on attachment: 211272 Committed r155507: <http://trac.webkit.org/changeset/155507>
All reviewed patches have been landed. Closing bug.