diff options
| author | 2019-03-26 13:26:17 -0700 | |
|---|---|---|
| committer | 2019-04-09 14:35:15 -0700 | |
| commit | ecf2e0f5cf0dcb2792bd4582f1c8e1e1ab2a365d (patch) | |
| tree | adb5c52e28293c3ea7e2efc9c90d745719d47917 /services/surfaceflinger/SurfaceFlinger.cpp | |
| parent | 381ac014109fcaed943a47395a65df64a01ea062 (diff) | |
Remove app vsync pausing
Bug: 128849305
Change-Id: I97c7ab9a2555f229783cd1c5352d883a627ab193
Test: jank tests
Test: systrace
Diffstat (limited to 'services/surfaceflinger/SurfaceFlinger.cpp')
| -rw-r--r-- | services/surfaceflinger/SurfaceFlinger.cpp | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp index bd3f156b82..6e05a19ad7 100644 --- a/services/surfaceflinger/SurfaceFlinger.cpp +++ b/services/surfaceflinger/SurfaceFlinger.cpp @@ -922,9 +922,6 @@ void SurfaceFlinger::setDesiredActiveConfig(const ActiveConfigInfo& info) { mDesiredActiveConfig.event = mDesiredActiveConfig.event | prevConfig; if (!mDesiredActiveConfigChanged) { - // This is the first time we set the desired - mScheduler->pauseVsyncCallback(mAppConnectionHandle, true); - // This will trigger HWC refresh without resetting the idle timer. repaintEverythingForHWC(); } @@ -1000,8 +997,6 @@ bool SurfaceFlinger::performSetActiveConfig() { // display is not valid or we are already in the requested mode // on both cases there is nothing left to do std::lock_guard<std::mutex> lock(mActiveConfigLock); - mScheduler->pauseVsyncCallback(mAppConnectionHandle, false); - mDesiredActiveConfig.event = Scheduler::ConfigEvent::None; mDesiredActiveConfigChanged = false; ATRACE_INT("DesiredActiveConfigChanged", mDesiredActiveConfigChanged); return false; @@ -1012,6 +1007,7 @@ bool SurfaceFlinger::performSetActiveConfig() { // Make sure the desired config is still allowed if (!isDisplayConfigAllowed(desiredActiveConfig.configId)) { std::lock_guard<std::mutex> lock(mActiveConfigLock); + mDesiredActiveConfig.event = Scheduler::ConfigEvent::None; mDesiredActiveConfig.configId = display->getActiveConfig(); return false; } |