Bug 156265

Summary: Test listbox clipping to contents rect
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, jonlee, simon.fraser, thorton, tonikitoo, webkit
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=128489
Attachments:
Description Flags
Patch
simon.fraser: review+, buildbot: commit-queue-
Archive of layout-test-results from ews122 for ios-simulator-wk2 none

Myles C. Maxfield
Reported 2016-04-05 17:07:18 PDT
Test listbox clipping to contents rect
Attachments
Patch (5.36 KB, patch)
2016-04-05 17:10 PDT, Myles C. Maxfield 		simon.fraser: review+
buildbot: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from ews122 for ios-simulator-wk2 (1.07 MB, application/zip)
2016-04-05 18:13 PDT, Build Bot 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2016-04-05 17:10:43 PDT
Created attachment 275723 [details] Patch
Simon Fraser (smfr)
Comment 2 2016-04-05 17:14:59 PDT
Comment on attachment 275723 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=275723&action=review > LayoutTests/fast/forms/listbox-padding-clip-overlay.html:17 > +<p>This test makes sure that the contents of list boxes do not intrude upon their padding. The test passes if you see a tall green box below (and not a single black pixel anywhere inside it)<p> > +<div style="display: inline-block; width: 50px; height: 400px; overflow: hidden;"> > +<select multiple="multiple" style="font: 50px Zapfino; margin: 0px; border: 0px solid black; padding: 0px; padding-left: 50px; background: green;"> > +<option>f</option><option>f</option><option>f</option><option>f</option><option>f</option> > +</select> > +</div><div style="display: inline-block; position: relative; width: 50px; height: 400px; overflow: hidden;"> > +<select multiple="multiple" style="font: 50px Zapfino; margin: 0px; border: 0px solid black; padding: 0px; padding-right: 50px; background: green; position: absolute; right: 0px; top: 0px;"> > +<option>f</option><option>f</option><option>f</option><option>f</option><option>f</option> This would be a lot neater with more html formatting and style up in the head.
Build Bot
Comment 3 2016-04-05 18:13:40 PDT
Comment on attachment 275723 [details] Patch Attachment 275723 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/1106335 New failing tests: fast/forms/listbox-padding-clip.html fast/forms/listbox-padding-clip-overlay.html
Build Bot
Comment 4 2016-04-05 18:13:43 PDT
Created attachment 275735 [details] Archive of layout-test-results from ews122 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews122 Port: ios-simulator-wk2 Platform: Mac OS X 10.10.5
Myles C. Maxfield
Comment 5 2016-04-05 18:17:24 PDT
Committed r199086: <http://trac.webkit.org/changeset/199086>
Note You need to log in before you can comment on or make changes to this bug.