[webkit-reviews] review granted: [Bug 23493] Introduce FormControlElementWithState abstraction : [Attachment 26955] Initial patch

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has granted Nikolas Zimmermann
<zimmermann at kde.org>'s request for review:
Bug 23493: Introduce FormControlElementWithState abstraction
https://bugs.webkit.org/show_bug.cgi?id=23493

Attachment 26955: Initial patch
https://bugs.webkit.org/attachment.cgi?id=26955&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
  if (!element->isFormControlElement())
 37	    return false;

Seems a bit strange when it's supposed to return a FormControlElement*

Strange indent:
FormControlElementWithState* toFormControlElementWithState(Element* element)
 59 {
 60	if (!element->isFormControlElementWithState())
 61		return 0;
 62

Tabs?
 {
260	 doc->registerFormElementWithState(this);
 260	FormControlElementWithState::registerFormControlElementWithState(this,
document());
261261 }

It's a little confusing that there are both member and free versions of the
toFormControlElement function.

I'm not sure what the EventHandlingElement changes are for.

In general this looks fine though.  Please be sure to fix the tabs/indent.