Kartikaya Gupta (email:kats@mozilla.staktrace.com) Reporter
	Description • 9 years ago

Spinoff from bug 1209970. Relevant bits quoted below:

(In reply to Mats Palmgren (:mats) from comment #21)
> The documentation for nsARefreshObserver says "When nothing needs to be
> painted, callers
> may not be notified".

(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #22)
> I think that documentation is out of date. My understanding is that now that
> we have DLBI and vsync we tick the refresh driver on vsyncs and always fire
> the refresh observers assuming we get through the other conditions at the
> top of nsRefreshDriver::Tick (and if those conditions cause a bailout then
> we don't need to dispatch a scroll event anyway).

The documentation in question was added with the original landing of nsRefreshDriver back in bug 435441 and is out of date.

	David Baron :dbaron:
	Comment 1 • 9 years ago

Hmmm.  That might have been intended to refer to things like the entire document being in a background tab.  If not, it's probably always been wrong.