Commit 6dd775f5 authored by Adam Jackson's avatar Adam Jackson

composite: Don't backfill non-bg-None windows

If there's a defined background then backfilling is a waste of effort,
since exposure processing will paint that in for us.  But note that we
have to backfill if any children are bg=None to preserve semantics with
non-composited servers.
Reviewed-by: default avatarKeith Packard <>
Signed-off-by: default avatarAdam Jackson <>
parent d2064fbb
......@@ -472,6 +472,17 @@ compUnredirectOneSubwindow (WindowPtr pParent, WindowPtr pWin)
return Success;
static int
bgNoneVisitWindow(WindowPtr pWin, void *null)
if (pWin->backgroundState != BackgroundPixmap)
if (pWin->background.pixmap != None)
static PixmapPtr
compNewPixmap (WindowPtr pWin, int x, int y, int w, int h)
......@@ -487,7 +498,18 @@ compNewPixmap (WindowPtr pWin, int x, int y, int w, int h)
pPixmap->screen_x = x;
pPixmap->screen_y = y;
* If there's no bg=None in the tree, we're done.
* We could optimize this more by collection the regions of all the
* bg=None subwindows and feeding that in as the clip for the
* CopyArea below, but since window trees are shallow these days it
* might not be worth the effort.
if (TraverseTree(pWin, bgNoneVisitWindow, NULL) == WT_NOMATCH)
return pPixmap;
if (pParent->drawable.depth == pWin->drawable.depth)
GCPtr pGC = GetScratchGC (pWin->drawable.depth, pScreen);
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment