Daniel Holbert f86b6c2086 Bug 1052471: Adjust test_bug453650.xhtml to perform its reflow-causing restyle a bit later, *after* the reflow observer is registered. r=jfkthame
I'm hoping this fixes (or at least helps with) this intermittent test failure
(an intermittent test-timeout).

Before this patch, the test did the following: it made a layout-impacting
restyle, and then it registered a reflow observer, and then it waited for the
previously-performed restyle to be flushed, which it then expects to cause a
reflow and trigger the reflow observer.

When the test times out and intermittently fails, it seems that the reflow
observer is successfully registered but never fires. I'm guessing that's
because the reflow is getting flushed eagerly for some reason, and happens
*before* the reflow observer is registered.  We can avoid this problem by
holding off on the restyle until after the reflow observer has been registered;
that's the approach that this patch takes.

(Note that this reflow-observer API isn't web-exposed; it's an internal API that
we use to report reflows in our devtools.)

Differential Revision: https://phabricator.services.mozilla.com/D100517
2021-01-04 19:04:40 +00:00
..