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

Commit d2121cf8 authored by Dan Stoza's avatar Dan Stoza Committed by android-build-merger
Browse files

Merge "SF: Fix repaint everything logic" into pi-dev 

am: 052a61eb

Change-Id: I44e04c4247a4b5b35962a77f7cc03cc601322045
parents 3d3e9966 052a61eb

services/surfaceflinger/SurfaceFlinger.cpp

+4 −15
Original line number Diff line number Diff line
@@ -1387,7 +1387,7 @@ void SurfaceFlinger::onRefreshReceived(int sequenceId, 
    if (sequenceId != getBE().mComposerSequenceId) {

        return;

    }

    repaintEverythingLocked();

    repaintEverything();

}



void SurfaceFlinger::setVsyncEnabled(int disp, int enabled) {
@@ -1949,7 +1949,7 @@ void SurfaceFlinger::setUpHWComposer() { 
    ALOGV("setUpHWComposer");



    for (size_t dpy=0 ; dpy<mDisplays.size() ; dpy++) {

        bool dirty = !mDisplays[dpy]->getDirtyRegion(false).isEmpty();

        bool dirty = !mDisplays[dpy]->getDirtyRegion(mRepaintEverything).isEmpty();

        bool empty = mDisplays[dpy]->getVisibleLayersSortedByZ().size() == 0;

        bool wasEmpty = !mDisplays[dpy]->lastCompositionHadVisibleLayers;
@@ -3716,7 +3716,7 @@ void SurfaceFlinger::setPowerModeInternal(const sp<DisplayDevice>& hw, 


        mVisibleRegionsDirty = true;

        mHasPoweredOff = true;

        repaintEverythingLocked();

        repaintEverything();



        struct sched_param param = {0};

        param.sched_priority = 1;
@@ -4637,22 +4637,11 @@ status_t SurfaceFlinger::onTransact( 
    return err;

}



void SurfaceFlinger::repaintEverythingLocked() {

void SurfaceFlinger::repaintEverything() {

    android_atomic_or(1, &mRepaintEverything);

    for (size_t dpy = 0; dpy < mDisplays.size(); dpy++) {

        const sp<DisplayDevice>& displayDevice(mDisplays[dpy]);

        const Rect bounds(displayDevice->getBounds());

        displayDevice->dirtyRegion.orSelf(Region(bounds));

    }

    signalTransaction();

}



void SurfaceFlinger::repaintEverything() {

    ConditionalLock _l(mStateLock,

            std::this_thread::get_id() != mMainThreadId);

    repaintEverythingLocked();

}



// A simple RAII class to disconnect from an ANativeWindow* when it goes out of scope

class WindowDisconnector {

public:

services/surfaceflinger/SurfaceFlinger.h

+0 −2
Original line number Diff line number Diff line
@@ -312,8 +312,6 @@ public: 


    // force full composition on all displays

    void repaintEverything();

    // Can only be called from the main thread or with mStateLock held

    void repaintEverythingLocked();



    // returns the default Display

    sp<const DisplayDevice> getDefaultDisplayDevice() const {