Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7b99776b authored by Baldev Sahu's avatar Baldev Sahu
Browse files

libgui: Copyback all region except dirty region for newly allocated buffer 

- Inside Surface::lock if new buffer is allocated and DirtyRegion does not
cover complete buffer bounds then copyback may not cover all remaining area
as it copyback only area covered by dirty regions from other buffers. This
will lead to left out black area which may cause flicker.

Change-Id: I4a3f7a56fc5fbaf4af926584919577d8d34bed57
parent 70f1cdcc

libs/gui/Surface.cpp

+7 −3
Original line number Diff line number Diff line
@@ -877,10 +877,14 @@ status_t Surface::lock( 
        if (canCopyBack) {

            Mutex::Autolock lock(mMutex);

            Region oldDirtyRegion;

            if(mSlots[backBufferSlot].dirtyRegion.isEmpty()) {

                oldDirtyRegion.set(bounds);

            } else {

                for(int i = 0 ; i < NUM_BUFFER_SLOTS; i++ ) {

                    if(i != backBufferSlot && !mSlots[i].dirtyRegion.isEmpty())

                        oldDirtyRegion.orSelf(mSlots[i].dirtyRegion);

                }

            }

            const Region copyback(oldDirtyRegion.subtract(newDirtyRegion));

            if (!copyback.isEmpty())

                copyBlt(backBuffer, frontBuffer, copyback);