RESOLVED FIXED Bug 49169
[Chromium] display:none has no effect on <option> element 
https://bugs.webkit.org/show_bug.cgi?id=49169
Summary [Chromium] display:none has no effect on <option> element
Shinichiro Hamaji
Reported 2010-11-08 04:13:58 PST
Bug 8351 is the master bug of this. This bug will fix only Chromium. Chromium side: http://code.google.com/p/chromium/issues/detail?id=29363
Attachments
Patch v1 (8.78 KB, patch)
2010-11-08 04:30 PST, Shinichiro Hamaji 		dglazkov: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Shinichiro Hamaji
Comment 1 2010-11-08 04:30:10 PST
Created attachment 73233 [details] Patch v1
Dimitri Glazkov (Google)
Comment 2 2010-11-08 10:13:13 PST
Comment on attachment 73233 [details] Patch v1 ok.
Shinichiro Hamaji
Comment 3 2010-11-09 02:41:00 PST
(In reply to comment #2) > (From update of attachment 73233 [details]) > ok. Thanks for your review! I've landed the patch as http://trac.webkit.org/changeset/71618 with modifying WebKit/chromium/ChangeLog as it had double ChangeLog entries.
Mihai Parparita
Comment 4 2010-11-09 07:35:49 PST
BTW, this broke PopupMenuTest (which lives upstream), though only the Windows build (it doesn't get built for other platforms). Should be fixed with http://trac.webkit.org/changeset/71631.
Shinichiro Hamaji
Comment 5 2010-11-09 07:49:50 PST
Oops. I thought I git-grepped with the symbol but I somehow missed it. Thanks for fixing this!
Eric Seidel (no email)
Comment 6 2010-12-14 01:43:37 PST
Looks like this was landed.
Note You need to log in before you can comment on or make changes to this bug.