[webkit-reviews] review granted: [Bug 64865] Switch isQuirkValue() virtual function to inline one. : [Attachment 110959] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted David Barr <davidbarr at chromium.org>'s
request for review:
Bug 64865: Switch isQuirkValue() virtual function to inline one.
https://bugs.webkit.org/show_bug.cgi?id=64865

Attachment 110959: Patch
https://bugs.webkit.org/attachment.cgi?id=110959&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
This idea is fine, but there is no reason to keep an entire class,
CSSQuirkPrimitiveValue, any more given that the class has no contents. Instead
we just need a new create function in CSSPrimitiveValue that creates a quirk
value.

Doing it that way will remove an entire unnecessary vtable, make the code
cleaner and easier to understand, and also remove the need to add new protected
members.

I’m going to say review+ but I’d much prefer this be redone to eliminate the
class.