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

Unified Diff: third_party/WebKit/LayoutTests/fast/events/select-event-on-input-recursive.html

Issue 2735633006: INPUT/TEXTAREA elements: Dispatch 'select' event only if text selection is changed. (Closed)
Patch Set: . Created 3 years, 9 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/events/select-event-on-input-recursive.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/select-event-on-input-recursive.html b/third_party/WebKit/LayoutTests/fast/events/select-event-on-input-recursive.html
deleted file mode 100644
index dcb15672305b7fa8d31bea327be4f335a0cc95d3..0000000000000000000000000000000000000000
--- a/third_party/WebKit/LayoutTests/fast/events/select-event-on-input-recursive.html
+++ /dev/null
@@ -1,32 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script src="../../resources/js-test.js"></script>
-</head>
-<body>
-<input id="input" value="abcd" />
-<script>
- window.jsTestIsAsync = true;
-
- var eventCalledCount = 0;
- input.addEventListener("select", function () {
- eventCalledCount++;
- input.selectionStart = 1;
- input.selectionEnd = 3;
yoichio 2017/03/07 09:00:56 This test confirms throttle event queueing. Could
tkent 2017/03/07 09:27:34 The throttling itself is unnecessary. I'll update
tkent 2017/03/07 11:30:08 Done. Uploaded new patch set.
yoichio 2017/03/08 02:00:36 Do you think this kind of flood is friends of infi
tkent 2017/03/08 02:05:23 This kind of flood won't happen after this CL beca
- });
-
- var animationFrameCount = 0;
- var animation = function () {
- animationFrameCount++;
- shouldBeTrue("animationFrameCount == eventCalledCount");
- if (animationFrameCount == 10)
- window.finishJSTest();
- else
- window.requestAnimationFrame(animation);
- }
- window.requestAnimationFrame(animation);
-
- input.select();
-</script>
-</body>
-</html>

Powered by Google App Engine
This is Rietveld 408576698