Bug 106273 - REGRESSION(r137406): Non-element nodes in <optgroup> hide later options
Summary: REGRESSION(r137406): Non-element nodes in <optgroup> hide later options
Status: RESOLVED DUPLICATE of bug 106298
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Elliott Sprehn
URL:
Keywords: Regression
Depends on:
Blocks:
 
Reported: 2013-01-07 16:20 PST by Elliott Sprehn
Modified: 2013-01-08 01:32 PST (History)
2 users (show)

See Also:


Attachments
Patch (4.02 KB, patch)
2013-01-07 16:49 PST, Elliott Sprehn
eric: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Elliott Sprehn 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>
Comment 1 Eric Seidel (no email) 2013-01-07 16:38:36 PST
trac.webkit.org/changeset/137406
Comment 2 Elliott Sprehn 2013-01-07 16:49:36 PST
Created attachment 181601 [details]
Patch
Comment 3 Eric Seidel (no email) 2013-01-07 16:50:34 PST
Comment on attachment 181601 [details]
Patch

LGTM.
Comment 4 Elliott Sprehn 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.
Comment 5 Elliott Sprehn 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 :)
Comment 6 Antti Koivisto 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 ***