Bug 1263458 - Force a full-page snapshot to make sure the scrollbars get included as well. r=botond a=test-only
authorKartikaya Gupta <kgupta@mozilla.com>
Wed, 14 Sep 2016 10:53:57 -0400
changeset 350230 22d3ca9e190a8967e02f7a7b2c7beaf8dcb3a0b4
parent 350229 657bba96bda0d2911948968c3664a64e22397758
child 350231 10046a00f401008f625c4b9ff6f0ca96f03e97b5
push id1230
push userjlund@mozilla.com
push dateMon, 31 Oct 2016 18:13:35 +0000
treeherdermozilla-release@5e06e3766db2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbotond, test-only
bugs1263458
milestone50.0a2
Bug 1263458 - Force a full-page snapshot to make sure the scrollbars get included as well. r=botond a=test-only The reftest-no-flush avoids some issues on Fennec where the reftest harness gets into an infinite loop of snapshotting because the pending-paint flag never gets cleared. The flag seems to be set as a result of the APZ flush and I'm unclear on why it doesn't get cleared normally, but this bypasses the problem. MozReview-Commit-ID: 8SGoSkRo5nm
layout/reftests/async-scrolling/disable-apz-for-sle-pages.html
--- a/layout/reftests/async-scrolling/disable-apz-for-sle-pages.html
+++ b/layout/reftests/async-scrolling/disable-apz-for-sle-pages.html
@@ -1,10 +1,10 @@
 <!DOCTYPE html>
-<html class="reftest-wait" reftest-async-scroll
+<html class="reftest-wait reftest-snapshot-all reftest-no-flush" reftest-async-scroll
       reftest-async-scroll-y="200">
  <head>
   <title>Check that the apz.disable_for_sle_pages pref behaves as expected</title>
   <script>
     // Upon load, this page scrolls to (0, 100). This triggers a scroll event,
     // which runs the scroll listener below. The scroll listener updates the
     // position of the div to simulate position:fixed using a scroll-linked
     // effect. The scroll-linked effect detector should then report that the