Re: offscreen animated images and doing repaint work

"L. David Baron" <[email protected]>
Newsgroups gmane.comp.mozilla.devel.layout
Message-ID <[email protected]>
On Thursday 2014-11-06 14:16 +1300, Robert O'Callahan wrote:
> On Thu, Nov 6, 2014 at 1:56 PM, L. David Baron <[email protected]> wrote:
> 
> > But each tick of the animated image (which I believe is a background
> > image) sends us through ImageLoader::DoRedraw, which in turn calls
> > nsIFrame::SchedulePaint, so that we do display list analysis to see
> > if anything changed.  (Nothing has, since the image is well
> > offscreen.)  But constructing and comparing the display lists takes
> > a good bit of time, leading to twitter using a good bit of CPU when
> > it's being displayed.
> >
> > Is it reasonable to test that the image is offscreen (or, really,
> > not in the region that's painted into a layer) somewhere earlier in
> > the process here?  Or is there something else we should be doing to
> > avoid this?
> >
> 
> Definitely. I think we can avoid the SchedulePaint if
> InvalidateImagesCallback doesn't find anything to invalidate. Got a bug
> number?

https://bugzilla.mozilla.org/show_bug.cgi?id=1094535

-David

-- 
𝄞   L. David Baron                         http://dbaron.org/   𝄂
𝄢   Mozilla                          https://www.mozilla.org/   𝄂
             Before I built a wall I'd ask to know
             What I was walling in or walling out,
             And to whom I was like to give offense.
               - Robert Frost, Mending Wall (1914)

_______________________________________________
dev-tech-layout mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-tech-layout
signature.asc (application/pgp-signature, 819 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBCAAGBQJUWtI+AAoJEO/hYSUPhPwhiasQAJSkn9bVMRJRVrQRgSSrMcsV
nT1PnTgwfoOZP4obI9+eZup0R4azkU/lNE/KQ9pVTlSu1GpVeypzk74wdx5j87a7
LtPe1w6F10nh7uCRtYsXdFaK3Qyh3s1SzsNUD7WGwCA2v2mvcTY1mkeVAbUw+ZUC
A1U2ASIsg3zRSfyrgRNnZ+s2s9BzzbgXo1tQPUC9ssnhuSTOEh9j+1qGVBrBZQeQ
OfJeEIMFSbBVDou1MZZfl+ZoVs1boRUnegF2pWRM6uNPCKouxJvoPKE80Ms/t1OV
7Cb7+lYm0ZOceJInxKJjv87p+B7FPPaZ27888vEVvdbHwnoWDf385teJMRnPWQfX
DeIHksLqXVIfayu2j1uDqYV0jX1+EE6fZJo3Ar6ff3FynYbSC8ss5R7r4yyTHBjh
4Z5rZtFxQc1HqW2131m+T3H51gstopGz4MzjMBcps8upFGyY7cgZ08EWm9uOORwF
v7G43aZvz5YZuFOAR158Gb6oYftTmlkF3is9FVm4KFW0JJNgcJHEdDmM5SLMPlIZ
g1BFbAWlACYSjLN4S06htStOJp2z+CJzXEEAKUyX8JXdBK5AsLUwjMA3I1AX76Rf
PRchXc5YIR2OQB4xt8Pw1sz2+Wbp49OuZdTSLbnhKOqZ1Y3i3qP/g8v89/ol3V6e
xO5bCUwBICfU2jdCCYhJ
=Hr0f
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.