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

Unified Diff: third_party/WebKit/LayoutTests/fast/forms/text/input-readonly-autoscroll.html

Issue 2536083002: SelectionAutoscroll should not happen in user-select:none. (Closed)
Patch Set: To pass the tests. Created 4 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/text/input-readonly-autoscroll.html
diff --git a/third_party/WebKit/LayoutTests/fast/forms/text/input-readonly-autoscroll.html b/third_party/WebKit/LayoutTests/fast/forms/text/input-readonly-autoscroll.html
index f8ef7b87dd5b5a773e89fabf9c54dfb2156afb3c..8106051cc0ee54412e70e254c3e82a9be34c2af9 100644
--- a/third_party/WebKit/LayoutTests/fast/forms/text/input-readonly-autoscroll.html
+++ b/third_party/WebKit/LayoutTests/fast/forms/text/input-readonly-autoscroll.html
@@ -17,7 +17,7 @@
eventSender.dragMode = false;
eventSender.mouseMoveTo(20, h);
eventSender.mouseDown();
- eventSender.mouseMoveTo(20, h);
+ eventSender.mouseMoveTo(40, h);
bokan 2016/12/05 15:39:36 Why was this change necessary?
sunyunjia 2016/12/05 15:44:39 Autoscroll can only be triggered AFTER some text h
bokan 2016/12/05 15:45:54 Yah, but wouldn't the mouseMove below cause select
sunyunjia 2016/12/05 16:25:53 I did some more investigation. The autoscroll is t
eventSender.mouseMoveTo(300, h);
}
setTimeout(autoscrollTestPart2, 100);

Powered by Google App Engine
This is Rietveld 408576698