Bug 106273

Summary:

REGRESSION(r137406): Non-element nodes in <optgroup> hide later options

Product:

WebKit

Reporter:

Elliott Sprehn <esprehn>

Component:

Layout and Rendering

Assignee:

Elliott Sprehn <esprehn>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

eric, koivisto

Priority:

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+	Details Formatted Diff Diff
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.