Skip to content

Commit 776ba5e

Browse files
Treehugger RobotAndroid (Google) Code Review
authored andcommitted
Merge "FrameTimeline: Optimize tracing information about frame start time" into main
2 parents 302cd56 + f24ce7c commit 776ba5e

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

services/surfaceflinger/FrameTimeline/FrameTimeline.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -801,6 +801,9 @@ void SurfaceFrame::traceActuals(int64_t displayFrameToken, nsecs_t monoBootOffse
801801
FrameTimelineDataSource::Trace([&](FrameTimelineDataSource::TraceContext ctx) {
802802
const auto timestamp = [&]() {
803803
std::scoped_lock lock(mMutex);
804+
if (mActuals.startTime != 0) {
805+
return mActuals.startTime;
806+
}
804807
// Actual start time is not yet available, so use expected start instead
805808
if (mPredictionState == PredictionState::Expired) {
806809
// If prediction is expired, we can't use the predicted start time. Instead, just
@@ -811,7 +814,7 @@ void SurfaceFrame::traceActuals(int64_t displayFrameToken, nsecs_t monoBootOffse
811814
return endTime - kPredictionExpiredStartTimeDelta;
812815
}
813816

814-
return mActuals.startTime == 0 ? mPredictions.startTime : mActuals.startTime;
817+
return mPredictions.startTime;
815818
}();
816819

817820
if (filterFramesBeforeTraceStarts && !shouldTraceForDataSource(ctx, timestamp)) {

0 commit comments

Comments
 (0)