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

Unified Diff: LayoutTests/fast/forms/input-placeholder-visibility-2-expected.html

Issue 258063005: Blink does not respect input.selectionStart and input.selectionEnd for some cases (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: New patch fixing existing bug Created 6 years, 7 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/input-placeholder-visibility-2-expected.html
diff --git a/LayoutTests/fast/forms/input-placeholder-visibility-2-expected.html b/LayoutTests/fast/forms/input-placeholder-visibility-2-expected.html
index 9616f61e90449fce97da1d3fd9783f9aea9db62b..363c5be0cff37ed9a79160a43263aebede48b964 100644
--- a/LayoutTests/fast/forms/input-placeholder-visibility-2-expected.html
+++ b/LayoutTests/fast/forms/input-placeholder-visibility-2-expected.html
@@ -7,7 +7,15 @@
<div>
<input id=i1 value="Text">
<script>
-document.getElementById('i1').setSelectionRange(4, 4);
+if (!window.testRunner) {
+ document.write("This test needs testRunner to work")
+} else {
+ var startX = document.getElementById("i1").offsetLeft + 30;
tkent 2014/06/01 23:40:55 This looks depending on a specific font, and looks
harpreet.sk 2014/07/02 10:13:05 Yes it will work. I had made the changes in new pa
+ var startY = document.getElementById("i1").offsetTop + 4;
+ eventSender.mouseMoveTo(startX,startY);
+ eventSender.mouseDown();
+ eventSender.mouseUp();
+}
</script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698