Event.h has too many virtual isMumbleEvent() functions
Created attachment 112006 [details] Patch
Comment on attachment 112006 [details] Patch Whoa, this is freakishly cool.
It doesn't quite compile yet, but soon.
Created attachment 112012 [details] Patch
Comment on attachment 112006 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=112006&action=review > Source/WebCore/bindings/js/JSErrorHandler.cpp:56 > + if (event->interfaceName() != eventNames().interfaceForErrorEvent) I think you want a .hasInterfaceName() instead. > Source/WebCore/bindings/objc/DOMEvents.mm:70 > + if (desiredInterface == eventNames().interfaceForMessageEvent) soooooo verbose.
(In reply to comment #5) > (From update of attachment 112006 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=112006&action=review > > > Source/WebCore/bindings/objc/DOMEvents.mm:70 > > + if (desiredInterface == eventNames().interfaceForMessageEvent) > > soooooo verbose. Yes. The actually-maintained bindings have this code autogenerated.
> I think you want a .hasInterfaceName() instead. I'm happy to add that, but I'd like to do that in a follow up patch.
Committed r98146: <http://trac.webkit.org/changeset/98146>