Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(57)

Unified Diff: third_party/WebKit/LayoutTests/fast/forms/select/menulist-height-change.html

Issue 2055593002: Reserve enough vertical space for the menuList layout element setting its logical height to the max… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: NeedsManualRebaseline Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/forms/select/menulist-height-change.html
diff --git a/third_party/WebKit/LayoutTests/fast/forms/select/menulist-width-change.html b/third_party/WebKit/LayoutTests/fast/forms/select/menulist-height-change.html
similarity index 52%
copy from third_party/WebKit/LayoutTests/fast/forms/select/menulist-width-change.html
copy to third_party/WebKit/LayoutTests/fast/forms/select/menulist-height-change.html
index c0e1291a45a7146780e838849bf482d8d036dd4a..e1492861e039a733960595f170c3dade23a95407 100644
--- a/third_party/WebKit/LayoutTests/fast/forms/select/menulist-width-change.html
+++ b/third_party/WebKit/LayoutTests/fast/forms/select/menulist-height-change.html
@@ -1,30 +1,36 @@
<html>
<head>
+ <meta http-equiv="content-type" content="text/html; charset=UTF-8">
+ <style type="text/css">
+ .big {
+ font-size: 70px;
+ }
+ </style>
<script>
function test() {
if (window.testRunner)
testRunner.waitUntilDone();
-
- setTimeout('addLongOption();', 100);
+
+ setTimeout('addLargeFontSizeOption();', 100);
}
- function addLongOption()
+ function addLargeFontSizeOption()
{
var newOpt = document.createElement("option");
- newOpt.appendChild(document.createTextNode("I require more room"));
+ newOpt.appendChild(document.createTextNode("Large Text"));
+ newOpt.className = "big";
var sl = document.getElementById("t");
sl.appendChild(newOpt);
-
+
if (window.testRunner)
testRunner.notifyDone();
}
</script>
</head>
<body onload="test()">
- This tests that when an option is dynamically added to a menu list, and it is too long for the current width,<br>
- that the select automatically recalculates the correct width.<br>
+ This tests that when an option is dynamically added to a menu list, and it is too big for the current height,<br>
+ that the select automatically recalculates the correct height.<br>
<select id="t">
- <option id="short">Short</option>
- <option id="medium">Medium</option>
+ <option>Normal text</option>
</select>
<div id="res"></div>
</body>

Powered by Google App Engine
This is Rietveld 408576698