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

Side by Side Diff: third_party/WebKit/LayoutTests/fast/events/resources/expect-cursor-update.js

Issue 2956023004: Add a flag to update hover effect when a layout is changed (Closed)
Patch Set: add hover flag Created 3 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 unified diff | Download patch
OLDNEW
1 // Wait until any mouse cursor update has completed, then verify the cursor info 1 // Wait until any mouse cursor update has completed, then verify the cursor info
2 // is what's expected and call the provided continuation. 2 // is what's expected and call the provided continuation.
3 // We need to poll to ensure EventHandler's cursor update timer has had a chance to fire. 3 // We need to poll to ensure EventHandler's cursor update timer has had a chance to fire.
4 function expectCursorUpdate(expectedInfo, completion) { 4 function expectCursorUpdate(expectedInfo, completion) {
5 // Need to give style application a chance to take effect first. 5 // Need to give style application a chance to take effect first.
6 requestAnimationFrame(function() { 6 requestAnimationFrame(function() {
7 // Note that cursorUpdatePending should (almost?) always be true at this 7 // Note that cursorUpdatePending should (almost?) always be true at this
8 // point, but we probably shouldn't depend on that in case scheduler cha nges 8 // point, but we probably shouldn't depend on that in case scheduler cha nges
9 // result in rAF not firing until after the cursor update timer as alrea dy 9 // result in rAF not firing until after the cursor update timer as alrea dy
10 // fired. 10 // fired.
11 var onFrame = function() { 11 var onFrame = function() {
12 if (internals.cursorUpdatePending) { 12 if (internals.cursorUpdatePending) {
13 requestAnimationFrame(onFrame); 13 requestAnimationFrame(onFrame);
14 } else { 14 } else {
15 shouldBeEqualToString('internals.getCurrentCursorInfo()', expect edInfo); 15 shouldBeEqualToString('internals.getCurrentCursorInfo()', expect edInfo);
16 completion(); 16 completion();
17 } 17 }
18 } 18 }
19 requestAnimationFrame(onFrame); 19 requestAnimationFrame(onFrame);
20 }); 20 });
21 } 21 }
22
23 function expectSendFakeMouseMove(expectedInfo, completion) {
24 // Need to give style application a chance to take effect first.
25 requestAnimationFrame(function() {
26 // Note that fakeMouseMovePending should (almost?) always be true at thi s
27 // point, but we probably shouldn't depend on that in case scheduler cha nges
28 // result in rAF not firing until after the fake_mouse_move_event_timer_ as
29 // already fired.
30 var onFrame = function() {
31 if (internals.fakeMouseMovePending) {
Navid Zolghadr 2017/07/05 17:37:33 Is there any reason we don't wait for cursorUpdate
lanwei 2017/07/06 03:04:34 Yes, they are different. When we sent fake mouse m
32 requestAnimationFrame(onFrame);
33 } else {
34 shouldBeEqualToString('internals.getCurrentCursorInfo()', expect edInfo);
35 completion();
36 }
37 }
38 requestAnimationFrame(onFrame);
39 });
40 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698