Skip to content

Commit

Permalink
Bug 1698439 - Replace calls to flushApzRepaints with promise equivale…
Browse files Browse the repository at this point in the history
…nt. r=tnikkel

These tests are too annoying to restructure completely, so I'm just
doing a local inlining of the function here.

Depends on D108438

Differential Revision: https://phabricator.services.mozilla.com/D108439
  • Loading branch information
staktrace committed Mar 16, 2021
1 parent 752487c commit 7f6293b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions dom/events/test/test_bug607464.html
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,9 @@
}
}, true);

flushApzRepaints(function() {
promiseApzRepaintsFlushed(win).then(function() {
scrollDown150PxWithPixelScrolling(scrollbox);
}, win);
});
}, win);
}

Expand Down
4 changes: 2 additions & 2 deletions gfx/layers/apz/test/mochitest/helper_long_tap.html
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@
if (eventsFired == 7) {
removeMouseEventListeners(target);
dump("Finished waiting for events, doing an APZ flush to see if any more unexpected events come through...\n");
flushApzRepaints(function() {
promiseApzRepaintsFlushed().then(function() {
dump("Done APZ flush, ending test...\n");
subtestDone();
});
Expand All @@ -82,7 +82,7 @@
removeMouseEventListeners(target);
synthesizeNativeTouch(target, 5, 5, SpecialPowers.DOMWindowUtils.TOUCH_REMOVE, function() {
dump("Finished synthesizing touch-end, doing an APZ flush to see if any more unexpected events come through...\n");
flushApzRepaints(function() {
promiseApzRepaintsFlushed().then(function() {
dump("Done APZ flush, ending test...\n");
subtestDone();
});
Expand Down

0 comments on commit 7f6293b

Please sign in to comment.