Merge "Don't double lock when resetting jank tests" into sc-dev

This commit is contained in:
TreeHugger Robot
2021-02-22 15:06:39 +00:00
committed by Android (Google) Code Review
2 changed files with 4 additions and 2 deletions

View File

@@ -236,7 +236,6 @@ void JankTracker::dumpFrames(int fd) {
}
void JankTracker::reset() {
std::lock_guard lock(mDataMutex);
mFrames.clear();
mData->reset();
(*mGlobalData)->reset();

View File

@@ -230,7 +230,10 @@ void RenderProxy::dumpProfileInfo(int fd, int dumpFlags) {
}
void RenderProxy::resetProfileInfo() {
mRenderThread.queue().runSync([=]() { mContext->resetFrameStats(); });
mRenderThread.queue().runSync([=]() {
std::lock_guard lock(mRenderThread.getJankDataMutex());
mContext->resetFrameStats();
});
}
uint32_t RenderProxy::frameTimePercentile(int percentile) {