systrace: forbid FrameEventHistory from stealing the fence
Previously if EGL_TIMESTAMPS_ANDROID is used, FrameEventHistory will
steal the ownership of the acquire fence after queueBuffer. The
FenceMonitor for tracing the GPU completion fence will crash in that
case.
Bug: 127781085
Test: install bouncyball.apk and do systrace without crashing
Change-Id: I6679324eb2a3b9366c957db4e9a14da5a87e98e8
diff --git a/libs/gui/Surface.cpp b/libs/gui/Surface.cpp
index 6460325..93b4191 100644
--- a/libs/gui/Surface.cpp
+++ b/libs/gui/Surface.cpp
@@ -816,7 +816,7 @@
// The consumer doesn't send it back to prevent us from having two
// file descriptors of the same fence.
mFrameEventHistory->updateAcquireFence(mNextFrameNumber,
- std::make_shared<FenceTime>(std::move(fence)));
+ std::make_shared<FenceTime>(fence));
// Cache timestamps of signaled fences so we can close their file
// descriptors.