commit b7d1230e46f1ed045ec2ed4c382f72cbb231b5f6
author Lloyd Pique <lpique@google.com> Thu Jul 26 12:56:47 2018 -0700
committer Lloyd Pique <lpique@google.com> Thu Jul 26 13:34:03 2018 -0700
tree 09c6efc14c5ccdf29f7f03888be0282e6b79e786
parent 854b8099ab21aeea3340c5f23239957b93016872

Revert "SF: Fix initialization of mRefreshStartTime"

This reverts commit 3d0957260a6e14fed746f126602a301d9be2b982.

Reason for revert: b/111793219 Power regression, with urgent fix needed.

Test: run cts -m CtsDeqpTestCases, run cts -m CtsViewTestCases

Change-Id: Ibd4c08b2be82a10c1e0a1a639b746e696ed8121c

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index df810c6..7da01c9 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -1765,8 +1765,6 @@
     ATRACE_CALL();
     ALOGV("preComposition");
 
-    mRefreshStartTime = systemTime(SYSTEM_TIME_MONOTONIC);
-
     bool needExtraInvalidate = false;
     mDrawingState.traverseInZOrder([&](Layer* layer) {
         if (layer->onPreComposition(mRefreshStartTime)) {
@@ -2220,6 +2218,8 @@
 
 void SurfaceFlinger::beginFrame()
 {
+    mRefreshStartTime = systemTime(SYSTEM_TIME_MONOTONIC);
+
     for (const auto& [token, display] : mDisplays) {
         bool dirty = !display->getDirtyRegion(mRepaintEverything).isEmpty();
         bool empty = display->getVisibleLayersSortedByZ().size() == 0;