Commit 4739d5a9 authored by Daniel Holbert's avatar Daniel Holbert
Browse files

Bug 1873763: Add 'overflow:hidden' to root scroller in some grid reftests, to...

Bug 1873763: Add 'overflow:hidden' to root scroller in some grid reftests, to avoid running afoul of scrollbar-presence nondeterminism.  a=diannaS

This is a workaround for an edge-case where the default 'overflow:auto' setup
can lead to a layout with or without a scrollbar, depending on whether the
first layout happens before vs. after the DOM is fully complete.  See
explanations in https://bugzilla.mozilla.org/show_bug.cgi?id=1873763#c0 and in
https://bugzilla.mozilla.org/show_bug.cgi?id=1873749 for more details. This
behavior seems to be common to all major browser engines, so it's probably not
worth worrying about too much, which is why I'm simply working around it in
these tests.

The changes to the -001 test files will address intermittent bug 1522442, and
the changes to the -010 and -011 test files are to address intermittent
bug 1809735.

Original Revision: https://phabricator.services.mozilla.com/D198080

Differential Revision: https://phabricator.services.mozilla.com/D202738
parent 500a3217
Loading
Loading
Loading
Loading
+2 −0
Original line number Diff line number Diff line
@@ -10,6 +10,8 @@
  <style type="text/css">
html,body {
    color:black; background-color:white; font-size:16px; padding:0; margin:0;
    /* We use overflow:hidden on root scroller to work around bug 1873749: */
    overflow: hidden;
}

.grid {
+2 −0
Original line number Diff line number Diff line
@@ -12,6 +12,8 @@
  <style type="text/css">
html,body {
    color:black; background-color:white; font-size:16px; padding:0; margin:0;
    /* We use overflow:hidden on root scroller to work around bug 1873749: */
    overflow: hidden;
}

.grid {
+2 −0
Original line number Diff line number Diff line
@@ -10,6 +10,8 @@
  <style type="text/css">
html,body {
    color:black; background-color:white; font-size:16px; padding:0; margin:0;
    /* We use overflow:hidden on root scroller to work around bug 1873749: */
    overflow: hidden;
}

.grid {
+2 −0
Original line number Diff line number Diff line
@@ -12,6 +12,8 @@
  <style type="text/css">
html,body {
    color:black; background-color:white; font-size:16px; padding:0; margin:0;
    /* We use overflow:hidden on root scroller to work around bug 1873749: */
    overflow: hidden;
}

.grid {
+2 −0
Original line number Diff line number Diff line
@@ -12,6 +12,8 @@
  <style type="text/css">
html,body {
    color:black; background-color:white; font-size:16px; padding:0; margin:0;
    /* We use overflow:hidden on root scroller to work around bug 1873749: */
    overflow: hidden;
}

.grid {