Bug 106273

Summary: REGRESSION(r137406): Non-element nodes in <optgroup> hide later options
Product: WebKit Reporter: Elliott Sprehn <esprehn>
Component: Layout and RenderingAssignee: Elliott Sprehn <esprehn>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: eric, koivisto
Priority: P2 Keywords: Regression
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch eric: review+

Elliott Sprehn
Reported 2013-01-07 16:20:11 PST
If the first child of an optgroup is a non-element then all later options in the select are missing. <select multiple size="3"> <optgroup>Text node</optgroup> <optgroup label="Third"> <option>After</option> </optgroup> </select>
Attachments
Patch (4.02 KB, patch)
2013-01-07 16:49 PST, Elliott Sprehn 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2013-01-07 16:38:36 PST
trac.webkit.org/changeset/137406
Elliott Sprehn
Comment 2 2013-01-07 16:49:36 PST
Created attachment 181601 [details] Patch
Eric Seidel (no email)
Comment 3 2013-01-07 16:50:34 PST
Comment on attachment 181601 [details] Patch LGTM.
Elliott Sprehn
Comment 4 2013-01-07 17:03:21 PST
The downstream bug is https://code.google.com/p/chromium/issues/detail?id=167598 , the report is because we broke bankofamerica.com's online baking website.
Elliott Sprehn
Comment 5 2013-01-07 17:03:58 PST
(In reply to comment #4) > The downstream bug is https://code.google.com/p/chromium/issues/detail?id=167598 , the report is because we broke bankofamerica.com's online baking website. Err, _banking_, not sure about their baking website :)
Antti Koivisto
Comment 6 2013-01-08 01:32:18 PST
This was already fixed in http://trac.webkit.org/changeset/139038. *** This bug has been marked as a duplicate of bug 106298 ***
Note You need to log in before you can comment on or make changes to this bug.