[webkit-reviews] review denied: [Bug 25345] Make EventListener::virtualisAttribute pure virtual : [Attachment 29712] Change EventListener::virtualisAttribute to be pure virtual, v1.

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has denied Dimitri Glazkov (Google)
<dglazkov at chromium.org>'s request for review:
Bug 25345: Make EventListener::virtualisAttribute pure virtual
https://bugs.webkit.org/show_bug.cgi?id=25345

Attachment 29712: Change EventListener::virtualisAttribute to be pure virtual,
v1.
https://bugs.webkit.org/attachment.cgi?id=29712&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
The overrides should be private, like the function in the base class.

Also, it's a programming mistake to call isAttribute on a subclass object, so
isAttribute should either be overridden to return false in those subclasses, or
be made private by putting "using EventListener::isAttribute" in the private
section of those classes.

Since this patch doesn't fix a bug, I'm going to say review- until you make the
overrides private.