summaryrefslogtreecommitdiff
path: root/services/surfaceflinger/SurfaceFlinger.cpp
diff options
context:
space:
mode:
author Alec Mouri <alecmouri@google.com> 2019-03-26 13:26:17 -0700
committer Alec Mouri <alecmouri@google.com> 2019-04-09 14:35:15 -0700
commitecf2e0f5cf0dcb2792bd4582f1c8e1e1ab2a365d (patch)
treeadb5c52e28293c3ea7e2efc9c90d745719d47917 /services/surfaceflinger/SurfaceFlinger.cpp
parent381ac014109fcaed943a47395a65df64a01ea062 (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.cpp6
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;
}