Bug 156265 - Test listbox clipping to contents rect
Summary: Test listbox clipping to contents rect
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-05 17:07 PDT by Myles C. Maxfield
Modified: 2016-04-06 12:15 PDT (History)
6 users (show)

See Also:


Attachments
Patch (5.36 KB, patch)
2016-04-05 17:10 PDT, Myles C. Maxfield
simon.fraser: review+
buildbot: commit-queue-
Details | Formatted Diff | Diff
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

Note You need to log in before you can comment on or make changes to this bug.
Description Myles C. Maxfield 2016-04-05 17:07:18 PDT
Test listbox clipping to contents rect
Comment 1 Myles C. Maxfield 2016-04-05 17:10:43 PDT
Created attachment 275723 [details]
Patch
Comment 2 Simon Fraser (smfr) 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.
Comment 3 Build Bot 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
Comment 4 Build Bot 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
Comment 5 Myles C. Maxfield 2016-04-05 18:17:24 PDT
Committed r199086: <http://trac.webkit.org/changeset/199086>