Change <option> white-space from pre to nowrap

When the option element is rendered, neither spaces nor line breaks are preserved in the text, so nowrap makes more sense than pre. Firefox currently has nowrap, whereas safari has pre. Bug: 561071 Change-Id: I5293453192b379761649de76eeb29130086747a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2587607 Commit-Queue: Joey Arhar <jarhar@chromium.org> Reviewed-by: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#838167}

Change <option> white-space from pre to nowrap
When the option element is rendered, neither spaces nor line breaks are preserved in the text, so nowrap makes more sense than pre. Firefox currently has nowrap, whereas safari has pre. Bug: 561071 Change-Id: I5293453192b379761649de76eeb29130086747a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2587607 Commit-Queue: Joey Arhar <jarhar@chromium.org> Reviewed-by: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#838167}
bcb42ec4 · Joey Arhar · Chromium LUCI CQ · 1d698f3c · bcb42ec4 · bcb42ec4
Commit bcb42ec4 authored Dec 17, 2020 by Joey Arhar Committed by Chromium LUCI CQ Dec 17, 2020
2 changed files
--- a/third_party/blink/renderer/core/html/resources/html.css
+++ b/third_party/blink/renderer/core/html/resources/html.css
@@ -847,7 +847,7 @@ option {
    font-weight: normal;
    display: block;
    padding: 0 2px 1px 2px;
-    white-space: pre;
+    white-space: nowrap;
    min-height: 1.2em;
 }

--- a/third_party/blink/web_tests/fast/forms/select/options-default-white-space.html
+++ b/third_party/blink/web_tests/fast/forms/select/options-default-white-space.html
+<!DOCTYPE html>
+<script src="../../../resources/testharness.js"></script>
+<script src="../../../resources/testharnessreport.js"></script>
+<select>
+  <option id=option value="hello">Hello</option>
+</select>
+<script>
+test(() => {
+  assert_equals(
+    window.getComputedStyle(option)['white-space'],
+    'nowrap');
+}, `<option>'s user agent 'white-space' should be 'nowrap'.`);
+</script>