[Webkit-unassigned] [Bug 50195] Progress and meter elements should take a form in their constructor like any other form control.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=50195





--- Comment #6 from Yael <yael.aharon at nokia.com>  2010-11-29 18:10:43 PST ---
(In reply to comment #2)
> (From update of attachment 75090 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=75090&action=review
> 
> review- because this bug does require a test case
> 
> > WebCore/ChangeLog:8
> > +        No new tests since this is a code cleanup only.
> 
> That is incorrect. The reason form elements get a form argument is so that they are correctly associated with the form in cases when the form is not an ancestor of the form element. We need to construct a test case that demonstrates this not occurring for meter elements before and happening correctly for meter elements after the patch.

Sorry, I thought that HTMLElement::findFormAncestor() was taking care of associating the correct form. I'll add a test.

I also need to touch some header file to force a rebuild on the buildbots.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.