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

Unified Diff: LayoutTests/fast/forms/select/popup-menu-appearance-tall.html

Issue 1075073004: New SELECT Popup: Fix window width to include scrollbar width (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Added TestExpectation entry Created 5 years, 8 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: LayoutTests/fast/forms/select/popup-menu-appearance-tall.html
diff --git a/LayoutTests/fast/forms/select/popup-menu-appearance-many.html b/LayoutTests/fast/forms/select/popup-menu-appearance-tall.html
similarity index 64%
copy from LayoutTests/fast/forms/select/popup-menu-appearance-many.html
copy to LayoutTests/fast/forms/select/popup-menu-appearance-tall.html
index 68424caa8fd888138e4a9c6f2ce691706a4bce27..7fc105791f08ad9fcd77eb66e6437787d820d287 100644
--- a/LayoutTests/fast/forms/select/popup-menu-appearance-many.html
+++ b/LayoutTests/fast/forms/select/popup-menu-appearance-tall.html
@@ -8,18 +8,18 @@ window.enablePixelTesting = true;
<script src="../resources/picker-common.js"></script>
</head>
<body>
-<select id="menu">
+<select id="menu" style="width: 10px; font-size: 300px;">
+ <option>X</option>
+ <option>X</option>
+ <option>X</option>
+ <option>X</option>
+ <option>X</option>
</select>
<p id="description" style="opacity: 0"></p>
<div id="console" style="opacity: 0"></div>
<script>
+description('Tests if width is increased for the scrollbar. PASS if the X is not clipped.');
var menu = document.getElementById('menu');
-for (var i = 0; i < 200; i++) {
- var option = new Option('item ' + i);
- if (i === 100)
- option.selected = true;
- menu.appendChild(option);
-}
openPicker(menu, finishJSTest, function () {
testFailed('picker didn\'t open')
finishJSTest();

Powered by Google App Engine
This is Rietveld 408576698