New SELECT Popup: Delay to create internal OPTION elements if there are many items.

New SELECT Popup: Delay to create internal OPTION elements if there are many items.

If the number of items is larger than ListPicker.DelayedLayoutThreshold(1000),
creates OPTION elements only for the first ListPicker.DelayedLayoutThreshold
items or this._config.selectedIndex before the first layout, then creates
remaining OPTIONs later.

Because it's possible that the popup width is shorter than the widest item, we
apply white-space:pre-wrap to OPTIONs in this case.

BUG=500401
R=keishi@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197800

[tkent, 2015-06-25 04:13:03]

tkent@chromium.org changed reviewers:
+ keishi@chromium.org

[tkent, 2015-06-25 04:13:04]

Keishi, please review this.

[keishi, 2015-06-25 04:38:00]

LGTM

https://codereview.chromium.org/1204163002/diff/1/Source/web/resources/listPi...
File Source/web/resources/listPicker.css (right):

https://codereview.chromium.org/1204163002/diff/1/Source/web/resources/listPi...
Source/web/resources/listPicker.css:16: .WRAP option {
nite: do we want to make this all caps?

[tkent, 2015-06-25 05:57:26]

https://codereview.chromium.org/1204163002/diff/1/Source/web/resources/listPi...
File Source/web/resources/listPicker.css (right):

https://codereview.chromium.org/1204163002/diff/1/Source/web/resources/listPi...
Source/web/resources/listPicker.css:16: .WRAP option {
On 2015/06/25 04:37:59, keishi wrote:
> nite: do we want to make this all caps?

Should be lowercased for consistency with other classes.
Done.

[tkent, 2015-06-25 05:57:32]

The CQ bit was checked by tkent@chromium.org

[tkent, 2015-06-25 05:57:32]

The patchset sent to the CQ was uploaded after l-g-t-m from keishi@chromium.org
Link to the patchset: https://codereview.chromium.org/1204163002/#ps20001
(title: "lower CSS class")

[tkent, 2015-06-25 07:04:34]

Committed patchset #2 (id:20001) manually as 197800 (presubmit successful).