Bug 49169

Summary: [Chromium] display:none has no effect on <option> element
Product: WebKit Reporter: Shinichiro Hamaji <hamaji>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, eric, mihaip
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Bug Depends on:    
Bug Blocks: 8351    
Attachments:
Description Flags
Patch v1 dglazkov: review+

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.