Bug 1285176 - fix intermittent test_mousecapture.xhtml by removing a check whose success we can't guarantee, r?kats draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Wed, 18 Jan 2017 16:10:49 +0000
changeset 463193 2380c3b52e98949fc9ed9d8e109206220a77ac16
parent 462769 80eac484366ad881c6a10bf81e8d9b8f7a676c75
child 542593 407dd525b7050549cb2b96739df3956f40850085
push id41974
push userbmo:gijskruitbosch+bugs@gmail.com
push dateWed, 18 Jan 2017 16:11:31 +0000
reviewerskats
bugs1285176
milestone53.0a1
Bug 1285176 - fix intermittent test_mousecapture.xhtml by removing a check whose success we can't guarantee, r?kats MozReview-Commit-ID: KzuOs4e9mCe
toolkit/content/tests/mochitest/test_mousecapture.xhtml
--- a/toolkit/content/tests/mochitest/test_mousecapture.xhtml
+++ b/toolkit/content/tests/mochitest/test_mousecapture.xhtml
@@ -69,22 +69,23 @@ function selectionScrollCheck() {
     otherWindow.removeEventListener("scroll", selectionScrollDone);
 
     var selectedText = otherWindow.getSelection().toString().replace(/\r/g, "");
     is(selectedText, "One\n\nTwo", "text is selected");
 
     // should have scrolled 20 pixels from the mousemove above and at least 6
     // extra 20-pixel increments from the selection scroll timer. "At least 6"
     // because we waited for 6 scroll events but multiple scrolls could get
-    // coalesced into a single scroll event. Therefore we allow 6-15 scrolls,
-    // which corresponds to a scroll position of 140 to 320. The 15 is chosen
-    // arbitrarily and can be adjusted.
+    // coalesced into a single scroll event, and paints could be delayed when
+    // the window loads when the compositor is busy. As a result, we have no
+    // real guarantees about the upper bound here, and as the upper bound is
+    // not important for what we're testing here, we don't check it.
     var scrollY = otherWindow.scrollY;
+    info(`Scrolled ${scrollY} pixels`);
     ok(scrollY >= 140, "selection scroll position after timer is at least 140");
-    ok(scrollY <= 320, "selection scroll position after timer is not more than 320");
     ok((scrollY % 20) == 0, "selection scroll position after timer is multiple of 20");
 
     synthesizeMouse(element, 4, otherWindow.innerHeight + 25, { type: "mouseup" }, otherWindow);
     disableNonTestMouseEvents(false);
     otherWindow.close();
 
     if (loadInWindow) {
       SimpleTest.finish();