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

Unified Diff: third_party/WebKit/LayoutTests/fast/forms/select-set-length-with-mutation-reorder.html

Issue 1509853008: Move select/option/optgroup-related tests in fast/forms to fast/forms/select. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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-set-length-with-mutation-reorder.html
diff --git a/third_party/WebKit/LayoutTests/fast/forms/select-set-length-with-mutation-reorder.html b/third_party/WebKit/LayoutTests/fast/forms/select-set-length-with-mutation-reorder.html
deleted file mode 100644
index 96120533d96102595e42d7c5f46d93e764667853..0000000000000000000000000000000000000000
--- a/third_party/WebKit/LayoutTests/fast/forms/select-set-length-with-mutation-reorder.html
+++ /dev/null
@@ -1,46 +0,0 @@
-<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
-<html>
-<head>
-<script src="../../resources/js-test.js"></script>
-</head>
-<body>
-<p id="description"></p>
-<div id="console"></div>
-<script>
-description('Tests that setting the .length on an HTMLSelectElement works in the presence of DOM mutation listeners that reorder option elements');
-
-var wrapper = document.createElement('div');
-document.body.appendChild(wrapper);
-wrapper.innerHTML = '<select id="theSelect">' +
- '<option id="a">a</option>' +
- '<option id="b">b</option>' +
- '<option id="c">c</option>' +
- '<option id="d">d</option>' +
- '</select>';
-
-var sel = document.getElementById('theSelect');
-
-var firstRemove = true;
-function onRemove(e) {
- if (firstRemove) {
- // remove listener temporarily to avoid lots of nesting
- sel.removeEventListener('DOMNodeRemoved', onRemove, false);
- var lastOption = document.getElementById('d');
- sel.removeChild(lastOption);
- sel.insertBefore(lastOption, document.getElementById('c'));
- firstRemove = false;
- sel.addEventListener('DOMNodeRemoved', onRemove, false);
- }
-}
-
-sel.addEventListener('DOMNodeRemoved', onRemove, false);
-sel.addEventListener('DOMNodeInserted', function() {}, false);
-
-shouldBe('sel.length', '4');
-sel.length = 2;
-shouldBe('sel.length', '2');
-shouldBe('sel.options.item(0).id', '"a"');
-shouldBe('sel.options.item(1).id', '"b"');
-</script>
-</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698