On Wed, 2010-05-05 at 15:45 -0700, Keith Packard wrote: > On Wed, 5 May 2010 16:25:28 -0400, Adam Jackson <[email protected]> wrote: > > > + > > + /* if we don't need to backfill, we're done */ > > + if (pWin->backgroundState != BackgroundPixmap) > > + return pPixmap; > > + if (pWin->background.pixmap != None) > > + return pPixmap; > > + > > Presumably this pixmap is being allocated for the window manager frame, > not the application itself. This isn't looking at the background for the > application, just the frame.
Oh, gross. If I'm reading this right, miPaintWindow clips by children (not IncludeInferiors, and called after map and validate), which means if you map an entire subtree at once, the bg=None regions poke through. Is that accurate? I'm not totally convinced that's required by the protocol, but I can imagine it being required by applications. It's easy enough to check, and the tree walk will almost certainly be faster than the blit. - ajax
signature.asc
Description: This is a digitally signed message part
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
