commit 7b570deea65bc7ac417a26ce3e4103bcef30e957
author John Reck <jreck@google.com> Mon Jun 27 13:27:23 2016 -0700
committer John Reck <jreck@google.com> Mon Jun 27 13:35:13 2016 -0700
tree 6e19b0c299e7bf7da395cddbe5f0d2384af5b28a
parent 157c6811de9bec883dfc802481bd07e64ca7ce59

Fix SurfaceViewPositionListener race bugs

Bug: 29628138

1: Make windowPositionLost synchronous as that's
what the Java side was expecting

2: Make the listener ref counted as otherwise
there's a race condition with the GC, which could
end up with use-after-frees

3: Ensure that all position updates are invoked
prior to frame completion

Change-Id: Iedbc017f611ba2878a49b4586612f79249ca2fe3

libs/hwui/renderthread/CanvasContext.cpp

diff --git a/libs/hwui/renderthread/CanvasContext.cpp b/libs/hwui/renderthread/CanvasContext.cpp
index d4956be..ceef9c7 100644
--- a/libs/hwui/renderthread/CanvasContext.cpp
+++ b/libs/hwui/renderthread/CanvasContext.cpp
@@ -784,6 +784,7 @@
     }
     sp<FuncTask> task(new FuncTask());
     task->func = func;
+    mFrameFences.push_back(task);
     mFrameWorkProcessor->add(task);
 }