LayoutTests/fast/events/touch/gesture/gesture-tap-reset-selection-range.html - Issue 1049233003: Keep the selection of the text field when changed by JS.

Unified Diff: LayoutTests/fast/events/touch/gesture/gesture-tap-reset-selection-range.html

Issue 1049233003: Keep the selection of the text field when changed by JS. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Created 5 years, 5 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

« LayoutTests/fast/events/touch/gesture/gesture-tap-input-after-composition.html ('K') | « LayoutTests/fast/events/touch/gesture/gesture-tap-input-after-composition.html ('k') | LayoutTests/fast/events/touch/gesture/gesture-tap-setrangetext-with-events.html » ('j') | LayoutTests/fast/forms/setrangetext-out-of-range.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/fast/events/touch/gesture/gesture-tap-reset-selection-range.html

diff --git a/LayoutTests/fast/events/touch/gesture/gesture-tap-reset-selection-range.html b/LayoutTests/fast/events/touch/gesture/gesture-tap-reset-selection-range.html

new file mode 100644

index 0000000000000000000000000000000000000000..b8035ad45a98b8075da25f3fc3581ec6dc32fed1

--- /dev/null

+++ b/LayoutTests/fast/events/touch/gesture/gesture-tap-reset-selection-range.html

@@ -0,0 +1,21 @@

+<!DOCTYPE html>

+<script src="../../../../resources/testharness.js"></script>

+<script src="../../../../resources/testharnessreport.js"></script>

+<div id="dragme">Make Selection Here</div>

+<script>

+var dragme = document.getElementById('dragme');

+if (window.eventSender) {

yosin_UTC9 2015/07/13 03:47:22 nit: You can move |if (window.eventSender) ...| to

Miyoung Shin(g) 2015/07/13 04:23:49 Done.

+ window.getSelection().selectAllChildren(dragme);

+ var x = dragme.offsetLeft + dragme.offsetWidth / 2;

+ var y = dragme.offsetTop + dragme.offsetHeight / 2;

+ eventSender.gestureTapDown(x, y);

+ eventSender.gestureShowPress(x, y);

+ eventSender.gestureTap(x, y);

+ test(function() {

+ assert_equals(window.getSelection().rangeCount, 0);

+ }, "This tests if the selection is reset after tapping of gesture.");

+if (window.testRunner)

yosin_UTC9 2015/07/13 03:47:22 nit: I think we don't need to hide HTML with w3c t

Miyoung Shin(g) 2015/07/13 04:23:49 If we remove this, we would need the expected resu

yosin_UTC9 2015/07/13 04:39:56 I don't think so. Other test files also have rende

tkent 2015/07/13 04:51:10 We need to add <div id=log></div> to hide the non-

Miyoung Shin(g) 2015/07/13 05:19:09 yosin@, tkent@, Thanks, I also found the document

+ dragme.style.display = "none";

+</script>