commit 19b4cf88e8ecdb40bfd56a0545be4beb8acee5b9
author Ady Abraham <adyabr@google.com> Thu Mar 28 18:24:48 2019 -0700
committer Ady Abraham <adyabr@google.com> Thu Mar 28 18:25:16 2019 -0700
tree f152a315e6565c2f4177a9112314e03f9d86b7fb
parent dc705b9eafeb07326155a580e722c7dca8d61892

DispSync: mLastCallbackTime is set incorrectly

Do not set mLastCallbackTime if callback was not dispatched.

Test: Chrome scrolling on BLAST
Change-Id: I3704f813b7923e6b4d72eb2ded5afbd7e3fe7758

services/surfaceflinger/Scheduler/DispSync.cpp

diff --git a/services/surfaceflinger/Scheduler/DispSync.cpp b/services/surfaceflinger/Scheduler/DispSync.cpp
index a2a6bd8..f72aef1 100644
--- a/services/surfaceflinger/Scheduler/DispSync.cpp
+++ b/services/surfaceflinger/Scheduler/DispSync.cpp
@@ -333,7 +333,6 @@
             if (t < now) {
                 if (isShorterThanPeriod(now - eventListener.mLastCallbackTime)) {
                     eventListener.mLastEventTime = t;
-                    eventListener.mLastCallbackTime = now;
                     ALOGV("[%s] [%s] Skipping event due to model error", mName,
                           eventListener.mName);
                     continue;