[Webkit-unassigned] [Bug 26141] Implement onformchange and onforminput event handlers
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jan 18 17:27:25 PST 2011
https://bugs.webkit.org/show_bug.cgi?id=26141
--- Comment #23 from Kent Tamura <tkent at chromium.org> 2011-01-18 17:27:24 PST ---
(From update of attachment 79358)
View in context: https://bugs.webkit.org/attachment.cgi?id=79358&action=review
> Source/WebCore/html/HTMLFormElement.cpp:601
> + if (!m_associatedElements[i]->isFormControlElement() || !m_associatedElements[i]->isResettable())
You shouldn't check isFormControlElement() here. We might have resettable non-HTMLFormControlElement classes in the future.
> Source/WebCore/html/HTMLFormElement.cpp:607
> + HTMLFormControlElement* formElement = static_cast<HTMLFormControlElement*>(elements[i].get());
Yes, we can cast elements[i] to HTMLFormControlElement unconditionally because a resettable control is always HTMLFormControlElement for now. But please add ASSERT(elements[i]->isFormControlElement()) before this line and add a comment about a reason why we don't need to handle non-HTMLFormControlElement case.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list