commit 12096d0562f9dd96ae3a54cb047faf7e041ca070
author Ana Krulec <akrulec@google.com> Fri Sep 07 14:54:14 2018 -0700
committer Ana Krulec <akrulec@google.com> Mon Sep 10 13:06:47 2018 -0700
tree 92f5a803cd1a43164b5da0243062fdcc61a5bd35
parent 2a268cf24096388d950da8809a6d9258f3dbfd06

SF: Moving Vsync modulator away from threads.

Modulator should not know about threads, just make a request to scheduler
whenever it's ready to update the offsets.

Bug: 113612090
Test: SF tests pass. Ran systrace -- offsetting works as expected.
Change-Id: Iaff11208ab8d1aa8a7c35a28e6802c1ca04e5243

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 55ef909..191651b 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -632,8 +632,8 @@
                                              });
 
         mEventQueue->setEventConnection(mScheduler->getEventConnection(mSfConnectionHandle));
-        mVsyncModulator.setEventThreads(mScheduler->getEventThread(mSfConnectionHandle),
-                                        mScheduler->getEventThread(mAppConnectionHandle));
+        mVsyncModulator.setSchedulerAndHandles(mScheduler.get(), mAppConnectionHandle.get(),
+                                               mSfConnectionHandle.get());
     } else {
         mEventThreadSource =
                 std::make_unique<DispSyncSource>(mPrimaryDispSync.get(),